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 #120 회원가입 시 email 추가 #120

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

hyxklee
Copy link
Member

@hyxklee hyxklee commented Feb 2, 2025

PR 내용

  • 소셜 회원가입시 email을 같이 받아서 저장하도록 수정했습니다.
  • 카카오 로그인 연동 중 jwt 토큰에 email이 없으면 문제가 발생하는 것을 발견했습니다.
  • 아직 미정이지만 다음 회의 때도 얘기해볼 문제로, email이 유지가 됐으면 좋겠다는 의견도 있었습니다.

PR 세부사항

  • 매퍼에서 email을 잘 매핑하고 있는 것을 확인 했습니다!

관련 스크린샷


주의사항

  • 아직 요구사항이 확정되진 않았으나, 변동될 가능성이 99프로라고 판단해 일단 작업 먼저 했습니당.

체크 리스트

  • 리뷰어 설정
  • Assignee 설정
  • Label 설정
  • 제목 양식 맞췄나요? (ex. #0 Feat: 기능 추가)
  • 변경 사항에 대한 테스트

@hyxklee hyxklee requested review from jj0526 and huncozyboy February 2, 2025 13:23
@hyxklee hyxklee self-assigned this Feb 2, 2025
@hyxklee hyxklee linked an issue Feb 2, 2025 that may be closed by this pull request
Copy link
Member

@huncozyboy huncozyboy left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

@@ -27,6 +27,7 @@ public record Register(
@NotNull Long kakaoId,
@NotBlank String name,
@NotBlank String studentId,
@NotBlank String email,
Copy link
Member

Choose a reason for hiding this comment

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

저번 회의 내용에서 회원가입할때 이메일 함께 입력받도록해서 변경 사항이 잘 반영된거 같아용

@hyxklee hyxklee merged commit 8559b9c into develop Feb 11, 2025
2 checks passed
@hyxklee hyxklee deleted the refactor/#118/이메일-추가 branch February 11, 2025 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor #118 소셜 회원가입 시 email 추가
2 participants