-
Notifications
You must be signed in to change notification settings - Fork 1
[Feat] 로그인 페이지 구현 #30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Note
|
| 코호트 / 파일 | 변경 요약 |
|---|---|
YAML 포맷 정규화 .github/ISSUE_TEMPLATE/feature.yml, .github/ISSUE_TEMPLATE/fix.yml, .github/ISSUE_TEMPLATE/refactor.yml, .github/workflows/ci.yml |
상위 수준 YAML 필드의 문자열 인용 형식을 이중 인용부호에서 단일 인용부호로 변경; feature.yml의 제목 값은 "[Feature]"에서 "[Feat]"로 단순화 |
로그인 페이지 리팩토링 src/pages/login/login-page.tsx, src/pages/login/login-page.css.ts |
컴포넌트명을 LoginPage에서 Login으로 변경; 기존 플레이스홀더 UI를 네비게이션, 상태 관리(useState), 스타일 기반 flex 레이아웃을 포함한 전체 로그인 폼으로 교체 |
신규 가입 페이지 src/pages/signup/signup-page.tsx |
기본 가입 페이지 컴포넌트 추가; 구조적 플레이스홀더만 포함 |
공유 컴포넌트 확장 src/shared/components/title/title.tsx, src/shared/components/title/title.css.ts, src/shared/components/button/button.css.ts |
Title 컴포넌트 신규 생성(children props 포함, xl 글자 크기, 굵은 가중치, 하단 여백); Button 컴포넌트의 secondary 변형 텍스트 색상을 blue200에서 blue300으로 변경 |
라우팅 구성 업데이트 src/shared/router/index.tsx, src/shared/router/path.ts |
SIGNUP 경로('/signup') 추가; SignupPage 컴포넌트를 라우터 진입점에 연결; 기존 경로 문자열 인용 형식을 단일 인용부호로 정규화 |
예상 코드 리뷰 소요 시간
🎯 2 (Simple) | ⏱️ ~10 분
검토 시 주목할 영역:
- 로그인 페이지의
useNavigate및useState구현이 의도한 네비게이션 흐름과 상태 관리 패턴을 따르는지 확인 - Title 컴포넌트의 CSS 스타일(레이아웃, 간격, 타이포그래피)이 전체 디자인 시스템과 일관성 있는지 검증
- 라우터 설정에서 SIGNUP 경로 정의 및 컴포넌트 매핑이 기존 라우팅 구조와 정렬되어 있는지 확인
📜 Recent review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Free
📒 Files selected for processing (12)
.github/ISSUE_TEMPLATE/feature.yml(1 hunks).github/ISSUE_TEMPLATE/fix.yml(1 hunks).github/ISSUE_TEMPLATE/refactor.yml(1 hunks).github/workflows/ci.yml(2 hunks)src/pages/login/login-page.css.ts(1 hunks)src/pages/login/login-page.tsx(1 hunks)src/pages/signup/signup-page.tsx(1 hunks)src/shared/components/button/button.css.ts(1 hunks)src/shared/components/title/title.css.ts(1 hunks)src/shared/components/title/title.tsx(1 hunks)src/shared/router/index.tsx(1 hunks)src/shared/router/path.ts(1 hunks)
Note
🎁 Summarized by CodeRabbit Free
Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.
Comment @coderabbitai help to get the list of available commands and usage tips.
📌 Related Issue
✅ Checklist
📚 Tasks
⭐ PR Point (To Reviewer)
📷 Screenshot
아무것도 입력하지 않았을 경우 버튼 활성화 x
아이디 비밀번호 모두 입력해야 활성화 o
Summary by CodeRabbit
릴리스 노트
새 기능
스타일
기타