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 추가 #185

Merged
merged 4 commits into from
Sep 3, 2023

Conversation

seunghaLim
Copy link
Collaborator

🍞 PR 타입

  • 기능 추가
  • 기능 수정
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

🍞 반영 브랜치

feature/#183 -> dev

🍞 변경 사항

  • 기존 카카오 로그인만 구현된 구조에 애플로그인을 적용했습니다
  • 기존 auth 관련 코드를 리팩토링하고 폴더링을 수정하였습니다
  • 소셜로그인 시 추가적으로 필요한 닉네임 변경 api를 추가했습니다

🍞 테스트 결과

🍞 To Reviewer

  • 소셜로 회원가입만 하고 닉네임 변경을 안한 회원에게 리프레시 토큰을 발급해주지 않는 로직이 고민입니다 추후에 의논해봅쉬다
  • 프로젝트 규모가 커지다보니 폴더링을 다시 한번 생각해봐야 될 것 같습니다
  • 리팩토링한 코드에 대해 같이 보면서 한번 더 수정해봐도 좋을 것 같습니다

@seunghaLim seunghaLim self-assigned this Sep 2, 2023
@seunghaLim seunghaLim changed the title Feature/#183 [FEAT] 애플 회원가입 추가 & 닉네임 변경 api 추가 Sep 2, 2023
Copy link
Member

@sung-silver sung-silver left a comment

Choose a reason for hiding this comment

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

제가 짠 로직까지 깔끔하게 수정해주셔서 감사합니다! 만나서 소셜로그인 시 혹은 같은 이메일로 가입한 유저인 경우에 어떻게 처리할지 이야기해보면 좋을 것 같습니다 ㅎㅎ

@seunghaLim seunghaLim merged commit 8d9253a into dev Sep 3, 2023
1 check passed
@seunghaLim seunghaLim deleted the feature/#183 branch February 17, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants