[Design] 비밀번호 재설정 뷰 GUI 반영 #321
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wuzoo
approved these changes
Nov 11, 2024
Comment on lines
+77
to
+83
| useEffect(() => { | ||
| handlePasswordMessage(form.updatedPassword); | ||
| }, [form.updatedPassword, handlePasswordMessage]); | ||
|
|
||
| useEffect(() => { | ||
| handlePasswordCheckerMessage(form.updatedPassword, form.updatedPasswordChecker); | ||
| }, [form.updatedPassword, form.updatedPasswordChecker, handlePasswordCheckerMessage]); |
There was a problem hiding this comment.
입력하는 password와 passwordChecker가 바뀔 때마다 supportingText도 그에 맞게 반응해야해서 불가피하게 useEffect를 사용했습니다. 다른 방법이 있을까요 ..?
Bowoon1216
approved these changes
Nov 12, 2024
rtttr1
approved these changes
Nov 12, 2024
| mutationFn: () => reSendEmail(email), | ||
| mutationFn: () => { | ||
| return reSendEmail(email); | ||
| }, |
There was a problem hiding this comment.
두 코드가 똑같이 동작할것 같은데 어떤점 때문에 return을 해주는 형식으로 바꾼거죠?
There was a problem hiding this comment.
저렇게 리턴을 해줘야 컴포넌트 내에서 .mutate로 접근이 가능하더라구요 !
| codeSupportingText, | ||
| setCodeSupportingText, | ||
| }; | ||
| }; |
There was a problem hiding this comment.
재사용이 되는 훅인가요? 제가 못 찾은 건지 한번 밖에 사용되지 않는 것 같은데 훅으로 빠져있어서 단순하게 가독성을 위해 분리한건지 궁금하여 여쭈어봅니다!
There was a problem hiding this comment.
재사용성 보다는 가독성을 위해서 분리한 게 더 크긴 합니다 !!
부가적 설명을 더하자면 code 인풋의 supportingText와 이메일 입력 인풋의 supportingText가 달라서 이 둘을 따로 관리해줬고 둘 모두 supportingText에 대한 상태를 관리해주기 때문에 일단 커스텀 훅으로 분리했습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
해당 이슈 번호
closed #320
체크리스트
💎 PR Point
모든 인풋에 supportingText가 있는 방식으로 뷰가 바뀌어서 해당 값들을 관리해줄 수 있는 커스텀 훅을 만들었습니다.
type은
default,success,error가 있으며, 이메일 인증을 거치는PasswordAuthPage의 경우는 mutation 함수 에러 여부에 따라 supportingText를 변경해줘야 해서 set함수를 페이지 내에서 사용해주었습니다.비밀번호 재설정
PasswordResetPage에서는 mutate 함수 성공 여부에 따라 에러 상태를 변경해주는 게 아니라서 커스텀 훅 내에서 set함수로 상태를 재설정해주었습니다.그리고 SupportingText 컴포넌트에서 기존에는 isNotice, isError의 prop이 있었으나, 맥락상 초록색은 성공 했을 때를 지칭하고 있어서
isNotice->isSuccess로 변경했습니다 !📌스크린샷 (선택)
2024-11-11.3.13.36.mov
2024-11-11.3.15.12.mov
2024-11-11.1.01.28.mov