-
Notifications
You must be signed in to change notification settings - Fork 2
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
[REFACTOR] react hook form register naming 통일 #177
Conversation
|
Deploying sopt-admin with Cloudflare Pages
|
ef36cf6
to
28e1cb1
Compare
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.
너무 좋습니다!! 감사합니다 🙇🏻♀️🙇🏻♀️🚀
✨ 구현 기능 명세
✅ PR Point
register 이름이 제 각각이라 관리하기 힘들더라고요
승희가 한 방식대로 객체로 관리하면 코드 중복도 많이 줄일 수 있을 거 같아 동일하게 처리해 줬어요
데이터 정상적으로 들어오는지와 에러 처리 잘 되는지만 체크 했어요
다른 기능들(e.g. Chip 모든 항목 required 처리 등)은 고려하지 않았습니다
코리할 때도 그 부분들만 보시면 될 거 같아요
변경한 부분
공통 탭 - 모집 일정, 브랜딩 컬러
지원하기 탭 - 파트별 소개, 자주 묻는 질문
데이터는 정상적으로 들어오는 걸 확인할 수 있었어요
입력 안 했을 때 에러 처리도 정상적으로 되고 있습니다
추후 해야할 것들