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] 소셜 로그인 멤버 회원가입 API #38

Merged
merged 4 commits into from
Feb 5, 2024
Merged

[FEAT] 소셜 로그인 멤버 회원가입 API #38

merged 4 commits into from
Feb 5, 2024

Conversation

arinming
Copy link
Member

@arinming arinming commented Feb 4, 2024

✨ PR 유형

어떤 변경 사항이 있나요??

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

🛠️ 작업내용

현재 로그인한 액세스 토큰을 기준으로 닉네임 설정 여부를 확인하고
설정이 되지 않았을 경우에 닉네임을 설정할 수 있도록 프론트와 협의하였습니다

📋 추후 진행 상황

닉네임 중복 확인 API

📌 리뷰 포인트

JwtTokenProvider로 현재 유저를 확인하도록 구현하였습니다

✅ Checklist

PR이 다음 요구 사항을 충족하는지 확인해주세요

  • 커밋 메시지 컨벤션에 맞게 작성했습니다
  • 유지-보수를 위해 주석 처리를 잘 작성했습니다

@arinming arinming added 🚶‍♀️아린 아린이 담당 ✨ feature 기능 추가 labels Feb 4, 2024
@arinming arinming self-assigned this Feb 4, 2024
Copy link
Contributor

@choeun7 choeun7 left a comment

Choose a reason for hiding this comment

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

회의 때 말씀하셨던 회원가입 api에 불필요한 email 삭제 등 꼼꼼하게 작성해주셨네요! 고생하셨습니다:)

Copy link
Contributor

@sanggae4133 sanggae4133 left a comment

Choose a reason for hiding this comment

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

고생하셨어요 꼼꼼하게 잘 구현하신 것 같습니다

Copy link
Contributor

@Jeongmin39 Jeongmin39 left a comment

Choose a reason for hiding this comment

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

확인했습니다! 수고하셨습니다 ~

@choeun7 choeun7 merged commit 90268cf into dev Feb 5, 2024
2 checks passed
@arinming arinming deleted the feature/35 branch March 10, 2024 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 기능 추가 🚶‍♀️아린 아린이 담당
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants