-
Notifications
You must be signed in to change notification settings - Fork 4
[✨feat] 캘린더 컴포넌트 생성 #40
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
|
@gummmmmy0v0 is attempting to deploy a commit to the projects Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
sohyun0
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.
캘린더 컴포넌트 정말 고생많으셨습니다~! 코멘트 확인 부탁드립니다.
또한, 과거 선택 제한 기능은 아직 안보이는것 같은데 추후 기능 확장하실 예정이실까요? 🤓
그리고 mode 관련해서 && 과 같은 반복적 로직이 많이보여서 도움을 드리려고했는데 관련내용 찾기가 어렵네요 ㅠ_ㅠ
header/nav.tsx 에 nav_config 로 저는 작성을 해서 최대한 분기를 줄이려고 했는데 상태랑 prop 까지는 어떻게 관리해야할지 저도 공부를 해봐야겠습니다 🥲
넵 과거 선택 제한 기능은 추후 확장 예정입니다! PR 올릴 때 후속 작업에 작성하는 걸 깜빡해서 다시 적어놨습니다! 캘린더가 작성할수록 복잡해지더라구요... 그래서 일단 잠깐 다른 작업 하고 후속 작업 하는 게 나을 것 같아서 PR 올렸습니다! |
92e8125 to
e544eab
Compare
a68b589 to
a0891ca
Compare
sohyun0
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.
고생하셨습니다!! 👏
중간중간에 const CALENDAR_ARROW_CLASS = clsx('rounded px-1 pt-1 hover:bg-gray-100'); 이런 상수들만 컴포넌트 밖으로 빼면 될것 같습니다 :)
추후 시간이 있으시다면 리팩토링 하실때 참고하시면 될것 같습니다 :)
📝 작업 개요 (필수)
✨ 작업 내용 (필수)
📸 스크린샷
🧐 해결해야 하는 문제
🤔 리뷰어 확인 필요 사항
🔗 관련 이슈
🛠️ 후속 작업
✅ 체크리스트 (필수)