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

Feat[#203] : 회원가입페이지 api 연결 #218

Merged
merged 8 commits into from
Mar 28, 2024

Conversation

grapefruit13
Copy link
Contributor

@grapefruit13 grapefruit13 commented Mar 28, 2024

관련 문서

유형

  • 기능 구현
  • UI 구현
  • 리팩토링
  • 버그 해결
  • 문서 업데이트
  • 기타( )

작업 내용

설명

📌 기능

1️⃣ useAuth.ts

  • 기존 useSignin.ts 를 useAuth.ts로 변경하고, 그 안에 useSignin 훅과 useSignup 훅을 담았습니다.
  • useMutation을 사용하여, mutate 사용시 onSuccess와 onError로 예외처리를 할 수 있도록 했습니다.

2️⃣ SignupForm/index.tsx

  • onSubmituseSignupmutate를 사용했습니다.
  • 회원가입 성공시 성공 모달을 보여주고, 성공 모달을 닫으면 로그인 페이지로 이동시킵니다.
  • 회원가입 실패시 에러 통신코드에 따른 실패 모달을 보여줍니다.
  • 중복 이메일일때의 에러 코드는 409이고, 잘못된 body 값일 때의 에러코드는 400입니다.

3️⃣ pages/signup/index.tsx

  • getServerSideProps에서 엑세스토큰을 확인 후, 엑세스토큰이 있으면(로그인 상태면) 롤 게시판으로 리다이렉트 시킵니다.

🎥 모션 영상

📸 디바이스별 스크린샷

@grapefruit13 grapefruit13 self-assigned this Mar 28, 2024
@grapefruit13 grapefruit13 added 기능 구현 기능 구현 UI 구현 UI 구현 labels Mar 28, 2024
@grapefruit13 grapefruit13 added this to the 페이지 API 연동 milestone Mar 28, 2024
Copy link
Contributor

@CheeseB CheeseB left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ccwnc ccwnc left a comment

Choose a reason for hiding this comment

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

LGTM

@grapefruit13 grapefruit13 merged commit eba1baf into dev Mar 28, 2024
@grapefruit13 grapefruit13 deleted the feat/signup-page/woorim branch March 28, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI 구현 UI 구현 기능 구현 기능 구현
Projects
Status: 👏 완료
Development

Successfully merging this pull request may close these issues.

Feat: 로그인, 회원가입 페이지 api 연동
4 participants