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

Refactor: 사용자 소셜 가입 정보 User 도메인 통합 #90

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

loveysuby
Copy link
Contributor

🚀 Related Issue

close: #88

📌 Tasks

  • 로그인 기능에서 사용되는 UserSocialAccount 도메인을 삭제합니다.

📝 Details

  • Refactor: 사용자 중복 OAuth 로그인 검증 로직 #86 에서 구현한 UserSocialAccount는 providerSerial 단일 필드와 동일한 기능을 수행합니다.
  • User 로그인 요청에 따른 가입 여부를 검증할 때마다 확인하는 정보이므로 User 도메인으로 병합합니다.

📚 Remarks

Points or opinions to share teams

@loveysuby loveysuby added the ♻️ refactor Refactoring code or architecture label Sep 30, 2024
@loveysuby loveysuby self-assigned this Sep 30, 2024
@loveysuby loveysuby linked an issue Sep 30, 2024 that may be closed by this pull request
@loveysuby loveysuby merged commit 6661f21 into develop Sep 30, 2024
1 check passed
@loveysuby loveysuby deleted the refactor/88 branch September 30, 2024 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor Refactoring code or architecture
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] UserSocialAccount User 도메인 통합
1 participant