-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
daepan
requested changes
Jun 8, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
한번 확인해주세요!
daepan
approved these changes
Jun 9, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다.
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What is this PR? 🔍
Changes 📝
이제 비밀번호 찾기를 문자 인증을 통해 진행합니다.
아직 전화번호를 통한 회원가입 페이지가 완성되지 않아서 전화번호 회원가입은 스웨거를 통해 해야한다는 불편함이 있습니다.
@hoooooony 님은 제 pr 머지된 후 SMS 브랜치 머지 받고 작업 마무리하시면 될 것 같습니다.
회원가입을 위한 api를 호출하기 위해 완료 버튼이 있는 CommonLayout 컴포넌트에 함수를 호출하고 사용하시면 됩니다.
ScreenShot 📷
Test CheckList ✅
Precaution
✔️ Please check if the PR fulfills these requirements
develop
branch unconditionally?main
?yarn lint