[Feat] 로그인, 회원가입 페이지 제작 #88
Conversation
- border-none 삭제 - getHintMessage 구조 단순화 - button 활성화 오류 수정
🎨 Storybook Reportℹ️ Story 변경사항이 감지되지 않았습니다 이 PR에는 Story 변경이 없어서 빌드를 스킵했습니다.
|
📊 Coverage Report
📉 #88을 main에 병합하면 coverage가 Coverage 요약@@ Coverage Diff @@
## main #88 +/- ##
===========================================
- Coverage 34.91% 34.71% -0.20%
===========================================
Files 81 82 +1
Lines 2692 2708 +16
Branches 158 159 +1
===========================================
Hits 940 940 0
+ Misses 1752 1768 +16 영향받은 파일✅ 이 PR로 영향받은 파일이 없습니다 수정된 모든 파일이 현재 coverage를 유지했습니다. |
🚀 PR Preview Report✨ Build가 성공적으로 완료되었습니다. Preview에서 변경사항을 확인하세요.
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
| {({ canSubmit, isSubmitting }) => { | ||
| const disabled = !canSubmit || isSubmitting; | ||
| {({ canSubmit, isSubmitting, isPristine }) => { | ||
| const disabled = !canSubmit || isSubmitting || isPristine; |
There was a problem hiding this comment.
isPristine이 isDirty의 반대군요 신기하네요
isPristine 조건은 혹시 모를 상황을 대비해 넣으신거라고 이해하면 될까요??
There was a problem hiding this comment.
그것도 있고, 맨 처음 페이지에 들어갔을 때 버튼이 비활성화 상태인게 고정되어 있었으면 해서 한 번 써봤습니다.
| const getHintMessage = (field: AnyFieldApi) => { | ||
| const { | ||
| meta: { errors, isTouched }, | ||
| } = field.state; | ||
| const { submissionAttempts } = field.form.state; | ||
|
|
📝 변경 사항
💡 components/pages/login/login-form/index.tsx
💡 components/pages/signup/signup-form/index.tsx
💡 components/shared/auth-switch-link/index.tsx
💡 app/signup/page.tsx
💡 app/login/page.tsx
🔗 관련 이슈
Closes #55 #57
🧪 테스트 방법
📸 스크린샷 (선택)
📋 체크리스트
💬 추가 코멘트
CodeRabbit Review는 자동으로 실행되지 않습니다.
Review를 실행하려면 comment에 아래와 같이 작성해주세요