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

[FE] fix: HomePage 비밀번호 입력에서 onBlur 이벤트가 일어나지 않으면 유효하지 않은 비밀번호라도 통과되는 버그 수정 #463

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

ImxYJL
Copy link
Contributor

@ImxYJL ImxYJL commented Aug 20, 2024


🚀 어떤 기능을 구현했나요 ?

  • 제목에 적힌 이슈를 수정했습니다

🔥 어떻게 해결했나요 ?

  • 원래 비밀번호는 에러 메세지가 있는지를 체크해서 유효성 검증을 했는데, 에러 메세지의 변화가 onBlur 이벤트에 의존적이었기 때문에 버그가 발생했습니다.
  • 그래서 별도의 비밀번호 검증 로직 함수를 이용해서 에러 메세지와 별개로 유효성을 검증해 URL 생성 버튼 활성화에 사용했습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

  • ㅎㅎ 하는김에 이것도 수정해놔야겠다 <- 섣부르게 수정하면 버그 생긴다.

Copy link
Contributor

@chysis chysis left a comment

Choose a reason for hiding this comment

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

잘 되는 것 직접 확인했어요! 🐎

Copy link
Contributor

@BadaHertz52 BadaHertz52 left a comment

Choose a reason for hiding this comment

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

🚀🚀🚀🚀

Copy link
Contributor

@soosoo22 soosoo22 left a comment

Choose a reason for hiding this comment

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

버그를 빠르게 찾아내고 해결해서 너무 멋있어요 올리!

@chysis chysis merged commit 8232634 into develop Aug 20, 2024
5 checks passed
@donghoony donghoony deleted the fe/fix/460-password-validation-button-disabled branch August 23, 2024 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
4 participants