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

[비즈니스] 비밀번호 찾기 문자 인증 도입 #348

Closed
wants to merge 16 commits into from

Conversation

chaeseungyun
Copy link
Contributor

  • Close #ISSUE_NUMBER

What is this PR? 🔍

  • 기능 : 비밀번호 찾기 문자 인증 도입
  • issue : #

Changes 📝

비밀번호 찾기할때 이메일 대신 문자로 본인 인증을 진행합니다.
전화번호는 하이픈이 없는 11자리 문자열입니다.

아직 문자 인증 회원가입이 만들어지지 않아서 전화번호 회원가입은 스웨거를 통해서 진행해야하는 불편함이 있습니다.

@hoooooony 님은 제 pr 머지되는 대로 pull 받고 진행하시면 될 것 같습니다.
마지막에 정보를 post하기 위한 함수를 다음 버튼이 있는 commonLayout에서 만들고 적용하시면 됩니다.

ScreenShot 📷

image
image
image
image

Test CheckList ✅

  • test 1
  • test 2
  • test 3

Precaution

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

chaeseungyun and others added 16 commits June 4, 2024 16:15
* feat: auth 단계를 조절하는 훅 추가

* feat: 비밀번호 찾기, 회원가입 공통 레이아웃 추가

* feat: 회원가입, 비밀번호 찾기 완성 페이지 추가

* feat: 공통 레이아웃 적용

* refactor: outlet에 props로 steps 전달

* fix: 오타 수정
* feat:  step 상태에 따라 button 색상 변경

* refactor: 이전 페이지로 돌아갈 때 isStepComplete 복구

* refactor: className 변경
@chaeseungyun chaeseungyun self-assigned this Jun 7, 2024
@github-actions github-actions bot requested review from hoooooony and KimKyungYun June 7, 2024 11:15
@chaeseungyun chaeseungyun deleted the feat/find_password_sms branch June 7, 2024 11:17
@chaeseungyun chaeseungyun restored the feat/find_password_sms branch June 7, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants