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

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

Merged
merged 18 commits into from
Jun 10, 2024

Conversation

chaeseungyun
Copy link
Contributor

  • Close #ISSUE_NUMBER

What is this PR? 🔍

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

Changes 📝

이제 비밀번호 찾기를 문자 인증을 통해 진행합니다.
아직 전화번호를 통한 회원가입 페이지가 완성되지 않아서 전화번호 회원가입은 스웨거를 통해 해야한다는 불편함이 있습니다.

@hoooooony 님은 제 pr 머지된 후 SMS 브랜치 머지 받고 작업 마무리하시면 될 것 같습니다.
회원가입을 위한 api를 호출하기 위해 완료 버튼이 있는 CommonLayout 컴포넌트에 함수를 호출하고 사용하시면 됩니다.

ScreenShot 📷

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 chaeseungyun self-assigned this Jun 7, 2024
@github-actions github-actions bot requested review from daepan and dooohun June 7, 2024 11:26
Copy link
Contributor

@daepan daepan left a comment

Choose a reason for hiding this comment

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

한번 확인해주세요!

src/page/Auth/FindPassword/ChangePassword/index.tsx Outdated Show resolved Hide resolved
src/page/Auth/FindPassword/ChangePassword/index.tsx Outdated Show resolved Hide resolved
src/page/Auth/FindPassword/ChangePassword/index.tsx Outdated Show resolved Hide resolved
src/page/Auth/FindPassword/Verify/index.tsx Outdated Show resolved Hide resolved
@chaeseungyun chaeseungyun requested a review from daepan June 8, 2024 08:44
Copy link
Contributor

@daepan daepan left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

@chaeseungyun chaeseungyun merged commit c271225 into feat/SMS Jun 10, 2024
@chaeseungyun chaeseungyun deleted the feat/find_password_sms branch June 10, 2024 02:18
hoooooony added a commit that referenced this pull request Jun 20, 2024
* [비즈니스] 회원가입, 비밀번호 찾기 공통 레이아웃 완성 (#341)

* feat: auth 단계를 조절하는 훅 추가

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

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

* feat: 공통 레이아웃 적용

* refactor: outlet에 props로 steps 전달

* fix: 오타 수정

* feat : 이전 회원가입 폴더명 변경, 동의 페이지 추가

* feat : 스테틱한 내용 파일 경로 수정

* feat : useStep에서 현재 단계 상태를 추가하여 다음 페이지 로직 통제 (#344)

Co-authored-by: (hoooooony) <([email protected])>

* 버튼 전역적으로 선언된 스타일 변경

* feat : 1페이지 ui 완성

* [비즈니스] step 상태에 따라 버튼 색상 변경 (#345)

* feat:  step 상태에 따라 button 색상 변경

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

* refactor: className 변경

* feat : 뒤로가기 대응

* fix : 동의상태관리 오류 수정

* [비즈니스] 비밀번호 찾기 문자인증 도입 (#349)

* feat: 비밀번호 찾기 초기 페이지 구성

* feat: 비밀번호 찾기 문자인증 api 생성

* feat: 문자 인증 response 인터페이스 추가

* feat: 비밀번호 찾기 문자 인증 페이지 완성

* refactor: 비밀번호 변경 api 생성 및 타입 생성

* feat: 비밀번호 변경 페이지 완성

* feat: 비밀번호 찾기 페이지 추가

* refactor: warning 아이콘 추가

* chore: scss 파일 위치 수정

* chore: 불필요한 파일 삭제 및 적용

* refactor: import cycle 해결

* fix: lint 에러 수정

* refactor: 사용자 입력으로 오류 수정 시 에러 초기화

* feat: useDebounce 훅 추가

* refactor: 오류 처리 및 useDebouce 적용

* refactor: 코드 가독성 향상

* feat : 스텝 2 추가

* feat : 컴포넌트 분리 및 스텝2 api 연결

* feat: step2 완성

* feat : 모달 추가

* feat  : modal 오류 수정

* refactor: 문자 인증 후 accessToken 저장

* feat: 다중 이미지 업로드 구현

* feat :step3 완성

* fix : 사업자 번호 오류 수정

* fix : 가게 선택 로직 오류 수정

* feat : 가게선택 css 추가

* feat: 전화번호 로그인 구현 및 적용

* fix: 경로 오타 수정

* fix: 디바운싱 추가 및 파라미터 변경

* feat : 오너 정보 등록 다음 단계 로직

* fix: 컨플릭트 해결

---------

Co-authored-by: chaeseungyun <[email protected]>
Co-authored-by: (hoooooony) <([email protected])>
Co-authored-by: chaeseungyun <[email protected]>
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