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

Feature(HK-70): 일반 회원가입 이메일 인증 페이지 구현 #13

Merged
merged 9 commits into from
Jan 4, 2025

Conversation

juiuj
Copy link
Contributor

@juiuj juiuj commented Jan 2, 2025

Motivation

  • Jira's HK-70 참고
  • 사용자가 이메일 인증 코드를 통해 유효성을 검증하여 회원가입을 진행하기 위한 페이지 구현

Problem Solving

  • 회원가입 페이지 router 설정(cb292c4)
  • 사용자로부터 이메일을 입력 받아 해당 이메일로 인증코드를 보낼 수 있는 페이지 구현(aa206a6)
  • previous, Next버튼을 통해 페이지 이동(aa206a6)

To Reviewer

  • Api 연결 후 따로 수정해야 할 부분은 TODO: 를 통해 기록해두었음. 추후 api 연결 시 수정 예정.
  • 이메일 인증 페이지만 우선 구현해뒀는데 다음 단계 페이지(Ex.비밀번호 설정, 약관 동의, 아이디/비번 찾기) 는 따로 브랜치를 파서 진행하는 건 어떨 지 의견 부탁드립니다!

@juiuj juiuj added enhancement New feature or request authorization labels Jan 2, 2025
@juiuj juiuj self-assigned this Jan 2, 2025
@juiuj juiuj changed the title Feature/hk 70 sign up page Feature(HK-69): 일반 회원가입 이메일 인증 페이지 구현 Jan 2, 2025
@juiuj juiuj requested a review from jinlee1703 January 2, 2025 17:12
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-13.d15ua1f8x2sllg.amplifyapp.com

Copy link
Contributor

@jinlee1703 jinlee1703 left a comment

Choose a reason for hiding this comment

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

P3: components/sign-up이라는 패키지 구조로 설정하게 된 경위가 궁금합니다!

src/components/sign-up/index.style.tsx Show resolved Hide resolved
@juiuj
Copy link
Contributor Author

juiuj commented Jan 3, 2025

P3: components/sign-up이라는 패키지 구조로 설정하게 된 경위가 궁금합니다!

sign-up 페이지 내에서도 이메일 인증 페이지, 비밀번호 찾기 등의 다양한 컴포넌트들이 구성될 예정이라 components/sign-up로 컴포넌트를 분리하였습니다!

jinlee1703
jinlee1703 previously approved these changes Jan 3, 2025
Copy link
Contributor

@jinlee1703 jinlee1703 left a comment

Choose a reason for hiding this comment

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

확인했습니다.

회원가입 각 스텝에 대해서는 별도의 PR로 쪼개서 작업하는 게 제가 리뷰할 때 흐름을 이해하기 수월할 것 같습니다!

고생하셨습니다 :)

src/components/sign-up/index.tsx Show resolved Hide resolved
@juiuj juiuj requested a review from jinlee1703 January 3, 2025 14:41
@juiuj juiuj changed the title Feature(HK-69): 일반 회원가입 이메일 인증 페이지 구현 Feature(HK-70): 일반 회원가입 이메일 인증 페이지 구현 Jan 3, 2025
Copy link
Contributor

@jinlee1703 jinlee1703 left a comment

Choose a reason for hiding this comment

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

확인했습니다. 고생하셨습니다 :)

@juiuj
Copy link
Contributor Author

juiuj commented Jan 4, 2025

확인감사합니다!

@juiuj juiuj merged commit f451eee into main Jan 4, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
authorization enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants