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

✨ 모든 로그인 & 회원가입 로직에서 성공 시 user pk 반환 #80

Merged
merged 2 commits into from
Jan 16, 2024

Conversation

psychology50
Copy link
Member

작업 이유

  • 프론트의 요청

작업 사항

  • 회원가입 및 oauth 로그인 & 회원가입 성공 시에도 userId를 반환합니다.
  • 응답 포맷은 login 시와 동일합니다.

이슈 연결

close #79

Copy link
Contributor

@heejinnn heejinnn left a comment

Choose a reason for hiding this comment

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

바로 만들어주셨군요 ㅎ 감사합니다!

@heejinnn heejinnn merged commit 055cbf7 into develop Jan 16, 2024
3 checks passed
@heejinnn heejinnn deleted the feat/79 branch January 16, 2024 16:44
@heejinnn
Copy link
Contributor

원래 기존 회원인 경우 oauth 로그인시 null값인 data를 돌려주셔서 그걸로 기존 회원 여부를 판단했었는데,
현재는 data에 userId를 돌려주시니 userId의 값이 있는지 없는지로 기존 회원 여부를 판단하면 될까요?

@psychology50
Copy link
Member Author

원래 기존 회원인 경우 oauth 로그인시 null값인 data를 돌려주셔서 그걸로 기존 회원 여부를 판단했었는데, 현재는 data에 userId를 돌려주시니 userId의 값이 있는지 없는지로 기존 회원 여부를 판단하면 될까요?

Login 시에는 userId 필드, 회원가입으로 넘어가야 할 때는 id 필드긴 한데
별도 검증 필드가 필요하시면 나중에라도 말씀해주세요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ Oauth 로그인 & 회원가입 시 User pk 반환
2 participants