-
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
REV-76/비밀번호 입력 컴포넌트 구현 #13
The head ref may contain hidden characters: "REV-76/\uBE44\uBC00\uBC88\uD638-\uCEF4\uD3EC\uB10C\uD2B8"
Conversation
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.
확실히 form 라이브러리를 쓰는 게 좋아보이네요!
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.
고생하셨습니다~!
저도 당연히 password와 confirm은 별도로 상태관리를 해야한다고 생각했습니다만, 많이 복잡진다면 form 라이브러리를 고려해봐도 좋을 것 같네요...!
src/components/PasswordInput.tsx
Outdated
onChange, | ||
...rest // 나머지 프로퍼티를 수집 | ||
}: PasswordInputProps) => { | ||
const handlePassWordChange = (event: ChangeEvent<HTMLInputElement>) => { |
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.
저희 팀 규칙으로 이벤트 객체 매개변수 명은 e
로 명시하기로 정했습니다! ㅎㅎ
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.
아앗...감사합니다
src/components/PasswordInput.tsx
Outdated
const handlePassWordChange = (event: ChangeEvent<HTMLInputElement>) => { | ||
onChange && onChange(event.target.value) | ||
} |
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.
onChange
함수를 외부에서 받고 있는데, handlePassWordChange를 따로 구현할 필요가 있을까요??
prop으로 받는 onChange 함수 명을 handlePassWordChange로 바꾸고, 바로 적용시켜줘도 깔끔할 것 같습니다!
📑 구현 내용
🚧 참고 사항
type
이 password면새 비밀번호 입력
용이고, confirmation이면새 비밀번호 확인
용도입니다.!구현을 했지만 문제점이 있을 거 같아요.
상태를 어떻게 관리할지..?
예를 들어 비밀번호 입력을 하고 다시 입력 확인을 받는 상태를
useState
로 관리하면 정말정말 피곤해질 거 같아요..!먼저
비밀번호 입력
과비밀번호 재검증
을 위한 상태를 2개 만들고 이것이 바뀔때마다 상태를 갱신해줘야 할 거 같아요또 이것이 일치한지 여부도 판별을 해야하는데 복잡해질거 같아서,
react-hook-form
과 같은 라이브러리를 쓰는 게 좋을 것 같은데 의논을 해봐야 할 것 같습니다!