-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat: #24 상태 추가 모달 UI 작성 및 Form Validation 추가 #30
Conversation
import Validator from '@utils/Validator'; | ||
import { deepFreeze } from '@utils/deepFreeze'; | ||
|
||
export const STATUS_VALIDATION_RULES = deepFreeze({ |
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.
freeze는 사용해봤는데 deepFreeze는 처음 들어봐서 찾아봤더니, 우석님이 직접 utils에 만들어두신 거였네요...! 우석님의 내공을 다시 한번 느끼면서... 많이 배워갑니다!
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.
이번 코드 작업하면서 고민을 많이 했다고 하셨는데, 그 고민의 흔적이 고스란히 느껴지는 코드네요. api 로직까지 완성되면 더 멋진 코드가 될 것 같아 기대됩니다! 참고해서 저도 더 공부해 보겠습니다.
예솔님 리뷰 감사합니다. 해결하고 보면 별 일이 아닌데, 고민할 땐 어렵게 다가오네요 ㅎㅎ.. 아직 추가적으로 구현해야할 기능들이 남아있지만, 1차적으로 UI 작업이 끝난 것 같습니다. 색상 추가에 대하여 한가지 고민이 있는데, 오늘 있을 회의에서 대화해보면 좋을 것 같아요. |
PR Type
What kind of change does this PR introduce?
Related Issues
#24 상태 추가 모달 UI 작성
What does this PR do?
Other information
View