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 분리 #165

Closed
wants to merge 4 commits into from
Closed

Conversation

ParkJuhan94
Copy link
Contributor

close #163

📑 작업 상세 내용

  • 프론트엔드에서 요구한 회원가입 api 분리를 위해서 관련된
    • entity, dto, controller, service, test 수정했습니다

💫 작업 요약

기존의 회원가입 api에서

  • 작업 1 Email / pw 등록과
  • 작업 2 나머지의 등록으로 분리

🔍 중점적으로 리뷰 할 부분

  • 회원가입 Api 를 분리하면서, step1 에서는 User 엔티티를 미리 만들고 (email/pw 제외한 필드는 null로 생성)
  • step2 에서 null인 필드들을 update 시켜줍니다.
  • User, Address 일부 필드의 nullable 속성을 true 로 변경했는데 문제가 없을지?
  • 통합 테스트는 잘 통과했습니다.

@ParkJuhan94 ParkJuhan94 added the feat New feature or request label May 22, 2024
@ParkJuhan94 ParkJuhan94 requested a review from hyun2371 May 22, 2024 17:35
@ParkJuhan94 ParkJuhan94 self-assigned this May 22, 2024
Copy link

Test Results

130 tests   129 ✅  27s ⏱️
 34 suites    1 💤
 34 files      0 ❌

Results for commit 0041bc9.

@hyun2371
Copy link
Member

고생 많으셨습니다:)

단일 회원가입 API를 credential과 profile API로 각각 분류하신 이유를 알 수 있을까요?

프론트 측에서는 oauth 때문에 API 분리를 요청했다고 하시던데
일반 회원가입 API, 소셜 로그인 API로 분리하면 되지 않나 싶어서요!

일반 회원가입 시에는 id, pw외에 프로필 사진, 닉네임, 주소, 선호 카테고리를 받는 걸로 알고 있습니다.
그러면 소셜 로그인 시에는 닉네임과 프로필 이미지를 카카오로부터 제공받고, 주소와 선호 카테고리만 추가 정보로 입력받으면 되지 않을까 생각이 듭니다.

정리하자면 제 생각에는 로직이 이렇게 구현되어야 할 것 같아요!
일반 회원

  • 기존 방법대로 로그인

소셜 회원

  • 소셜 로그인 요청 후 회원 가입
  • 추가 정보 입력 받아 회원 DB 업데이트

@hyun2371
Copy link
Member

User, Address 일부 필드의 nullable 속성을 true 로 변경했는데 문제가 없을지?

이 부분은 문제 없을 것 같습니다!

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

Successfully merging this pull request may close these issues.

[feat] : 회원가입 api 분리
2 participants