Skip to content
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

Merged
merged 5 commits into from
Oct 31, 2023

Conversation

khj0426
Copy link
Collaborator

@khj0426 khj0426 commented Oct 30, 2023

📑 구현 내용

  • 비밀번호를 입력받을 수 있는 컴포넌트를 만들었습니다.!

image


🚧 참고 사항

type이 password면 새 비밀번호 입력용이고, confirmation이면 새 비밀번호 확인용도입니다.!

interface PasswordInputProps {
 //새 비밀번호 입력 | 새 비밀번호 확인(재검증)
  type: 'password' | 'confirmation'
  onChange?: (newPassword?: string) => void
  disabled?: boolean
  placeholder?: string
}

구현을 했지만 문제점이 있을 거 같아요.

상태를 어떻게 관리할지..?

예를 들어 비밀번호 입력을 하고 다시 입력 확인을 받는 상태를 useState로 관리하면 정말정말 피곤해질 거 같아요..!
먼저 비밀번호 입력비밀번호 재검증을 위한 상태를 2개 만들고 이것이 바뀔때마다 상태를 갱신해줘야 할 거 같아요

또 이것이 일치한지 여부도 판별을 해야하는데 복잡해질거 같아서,

react-hook-form과 같은 라이브러리를 쓰는 게 좋을 것 같은데 의논을 해봐야 할 것 같습니다!

@khj0426 khj0426 added the 기능 새로운 기능을 개발합니다. label Oct 30, 2023
@khj0426 khj0426 self-assigned this Oct 30, 2023
@khj0426 khj0426 changed the title REV-76/질문 입력 컴포넌트 구현 REV-76/비밀번호 입력 컴포넌트 구현 Oct 31, 2023
Copy link
Member

@hyoribogo hyoribogo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확실히 form 라이브러리를 쓰는 게 좋아보이네요!

Copy link
Collaborator

@hayamaster hayamaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다~!
저도 당연히 password와 confirm은 별도로 상태관리를 해야한다고 생각했습니다만, 많이 복잡진다면 form 라이브러리를 고려해봐도 좋을 것 같네요...!

onChange,
...rest // 나머지 프로퍼티를 수집
}: PasswordInputProps) => {
const handlePassWordChange = (event: ChangeEvent<HTMLInputElement>) => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저희 팀 규칙으로 이벤트 객체 매개변수 명은 e로 명시하기로 정했습니다! ㅎㅎ

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아앗...감사합니다

Comment on lines 19 to 21
const handlePassWordChange = (event: ChangeEvent<HTMLInputElement>) => {
onChange && onChange(event.target.value)
}
Copy link
Collaborator

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로 바꾸고, 바로 적용시켜줘도 깔끔할 것 같습니다!

@khj0426 khj0426 merged commit 5f022cc into develop Oct 31, 2023
1 check failed
@khj0426 khj0426 deleted the REV-76/비밀번호-컴포넌트 branch October 31, 2023 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 새로운 기능을 개발합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants