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

[Feat] 비밀번호 재설정 QA #228

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from
Open

Conversation

namdaeun
Copy link
Member

@namdaeun namdaeun commented Sep 15, 2024

해당 이슈 번호

closed #226


체크리스트

  • 🔀 PR 제목의 형식을 잘 작성했나요? e.g. [feat] PR을 등록한다.
  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 💻 git rebase를 사용했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

📌 내가 알게 된 부분

  const [form, setForm] = useState({
    updatedPassword: '',
    updatedPasswordChecker: '',
  });
  • form에 updatedPasswordupdatedPasswordChecker를 함께 관리해주었고, 처음에는 input이 클릭되는 여부를 isFocused라는 상태로 또 관리해줬는데, form 상태 값이 있으면 Input의 isErrorSupportingText를 띄워주는 식으로 코드 작성해주었습니다.

📌 질문할 부분


📌스크린샷 (선택)

2024-09-15.7.44.26.mov
2024-09-15.7.45.01.mov

Copy link

🚀 Storybook 확인하기 🚀

Copy link

🚀 Storybook 확인하기 🚀

Copy link

🚀 Storybook 확인하기 🚀

Copy link

🚀 Storybook 확인하기 🚀

Copy link

🚀 Storybook 확인하기 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

QA 해결
1 participant