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

[#346] 이메일 비밀번호 유효성 검사 추가 #347

Merged
merged 5 commits into from
Jun 16, 2024

Conversation

uuuunseo
Copy link
Contributor

@uuuunseo uuuunseo commented Jun 15, 2024

💡 배경 및 개요

QA결과에서 로그인, 비밀번호 찾기, 비밀번호 변경, 회원가입 페이지에서 비밀번호, 이메일, 전화번호의 유효성검사가 진행되지 않고, 발생한 문제들을 알려주지 않고 있어서 불편하다는 문제가 있었어요.

Resolves: #346

📃 작업내용

  • 로그인 페이지에서 로그인 버튼 클릭시 유효성을 검사하도록 수정했어요.
  • 로그인 페이지에서 아직 승인되지 않은 계정으로 가입할 때 리프레시 토큰 오류메시지가 뜨는 문제를 해결했어요.
  • 비밀번호 찾기 페이지에서 이메일 유효성을 검사하도록 추가했어요.
  • 비밀번호 찾기 페이지에서 가입되지 않은 이메일로 요청을 보냈을 때 에러메시지를 추가했어요.
  • 비밀번호 변경 페이지에서 현재 비밀번호와 새 비밀번호가 똑같을 때 에러메시지를 띄우도록 추가했어요.
  • 비밀번호 변경 페이지에서 현재 비밀번호, 새 비밀번호의 유효성검사 기능을 추가했어요.
  • 회원가입 페이지에서 전화번호, 이메일, 비밀번호 유효성 검사를 추가했어요.

🙋‍♂️ 리뷰노트

마이페이지에서 로그아웃, 회원탈퇴 진행 시 리다이렉트 되는 코드가 추가되었는데 간단한 작업이라 이번 PR에 추가했어요.

✅ PR 체크리스트

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. .env, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "환경값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

@uuuunseo uuuunseo added 0️⃣ Priority: Critical 우선순위 - 긴급!!!!! ♻️ Refactor 코드 리팩토링 ⚡️ Simple 간단한 변경사항 ✨ Feature 신규 기능 🐞 Bug 버그 발생 labels Jun 15, 2024
@uuuunseo uuuunseo requested a review from Xixn2 June 15, 2024 13:17
@uuuunseo uuuunseo self-assigned this Jun 15, 2024
@uuuunseo uuuunseo linked an issue Jun 15, 2024 that may be closed by this pull request
Copy link
Contributor

@ani2689 ani2689 left a comment

Choose a reason for hiding this comment

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

LGTM

@uuuunseo uuuunseo merged commit d82dd7b into master Jun 16, 2024
2 checks passed
@uuuunseo uuuunseo deleted the 346-fix-regular-expression branch June 16, 2024 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0️⃣ Priority: Critical 우선순위 - 긴급!!!!! 🐞 Bug 버그 발생 ✨ Feature 신규 기능 ♻️ Refactor 코드 리팩토링 ⚡️ Simple 간단한 변경사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

이메일, 비밀번호 정규식 문제 해결
2 participants