-
Notifications
You must be signed in to change notification settings - Fork 0
[Fix]: 모임생성 날짜 필드에서 현재 시간 이상만 받도록 zod 유효성검사 수정 #313
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
|
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 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
📊 Coverage Report
📉 #313을 main에 병합하면 coverage가 Coverage 요약@@ Coverage Diff @@
## main #313 +/- ##
===========================================
- Coverage 39.57% 39.56% -0.01%
===========================================
Files 214 214 0
Lines 9562 9565 +3
Branches 411 411 0
===========================================
Hits 3784 3784 0
+ Misses 5778 5781 +3 영향받은 파일✅ 이 PR로 영향받은 파일이 없습니다 수정된 모든 파일이 현재 coverage를 유지했습니다. |
🎭 Playwright Report✨ E2E Test가 성공적으로 완료되었습니다. Test 요약 내용을 확인해주세요.
📊 Test Summary
📜 Test Details✅ Passed Tests (3)
|
🎨 Storybook Report✅ 변경 사항이 없습니다 모든 Story가 이전 빌드와 동일합니다.
|
🚀 PR Preview Report✨ Build가 성공적으로 완료되었습니다. Preview에서 변경사항을 확인하세요.
|
claudia99503
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
coerce가 input 등 form 요소에 입력한 특정 value를 내가 원하는 type으로 변환시켜서 상태 관리하고 싶을 때 사용하는 거..? 이게 맞나요
📝 변경 사항
[삽질]
이러한 이유로 string과 date간의 타입 충돌이 발생했고 결론적으로 coerce 계산을 버리고 기존 startTime의 schema 대로 string을 입력받은 후 refined 를 활용해 입력 받은 string을 직접 new Date(value) 로 변환해서 유효한 시간인지 검사함.
[수확]
🔗 관련 이슈
Closes #
🧪 테스트 방법
📸 스크린샷 (선택)
📋 체크리스트
💬 추가 코멘트
CodeRabbit Review는 자동으로 실행되지 않습니다.
Review를 실행하려면 comment에 아래와 같이 작성해주세요