Feat/#3

[ojy0903]

📌 관련 이슈

• Close feat: 회원가입 - 회원 도메인 설계 및 단순 회원가입 구현 #3

🚀 개요

User 엔티티 설계 및 단순 회원가입(이메일 인증 X) 로직 구현

📄 작업 내용

구체적인 작업 내용을 설명해주세요.

• User 엔티티 설계 및 레포지토리 적용
• 단순 회원가입 (DB 저장) 로직 설계
• SecurityConfig 모든 접근 허용하여 Swagger 접근 허용

📸 스크린샷 / 테스트 결과 (선택)

결과물 확인을 위한 사진이나 테스트 로그를 첨부해주세요.

1. SecurityConfig 아직 로그인 구현이 없으므로 모든 접근 허용, Swagger 정상 접근 확인

2. Postman 통해 회원 등록 성공

3. DB 정상 반영 및 비밀번호 암호화 저장

====오류 처리 ====
4. 이미 가입된 이메일로 회원가입 시도 시 오류(중복 이메일)

5. 필수 필드가 비어있을 경우(@Valid 오류)

✅ 체크리스트

• [✅] 브랜치 전략(GitHub Flow)을 준수했나요?
• [✅] 메서드 단위로 코드가 잘 쪼개져 있나요?
• [✅] 테스트 통과 확인
• [✅] 서버 실행 확인
• [✅] API 동작 확인

---

🔍 리뷰 포인트 (Review Points)

리뷰어가 중점적으로 확인했으면 하는 부분을 적어주세요. (P1~P4 적용 가이드)

• 피그마를 확인해보니까 회원가입 할 때 이메일 인증을 바로 진행하더라구요... 가입 한 이후에 이메일 인증하는거로 착각해버려서...
  일단 인증없는 지금 로직부터 머지 하고, 다음 이슈로 이메일 인증 로직을 회원가입 로직에 끼워넣는 방식으로 진행하겠습니다
• 아직 로그인 기능이 없어서 SecurityConfig 에서 모든 접근을 허용하는 방식으로 설정했습니다. Swagger 정상 접근도 확인했습니다.
• User 엔티티 설계가 적절한지 여부 확인 한번 해주시면 좋을 것 같습니다.
• DTO 에 관해서 Converter 클래스를 따로 두고 DTO 에서 엔티티로 매핑하는 방식을 사용했는데
  DTO 레코드 내부에 메서드를 넣는 방식으로 바꾸는게 나을까요?

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

[ojy0903]

공통 응답 부분이랑 같이 하니까 커밋이 좀 많아졌네요...ㅠㅠ 리뷰 하실때는

1. User 엔티티 설계 부분
2. SecurityConfig 접근 허용 부분
3. UserService 및 UserController
4. DTO 변환을 Converter 로 할지 or DTO record 내부 메서드로 할지

이 부분들을 중점적으로 보시면 될 것 같습니다

[jinnieusLab]

P4: 전반적인 코드는 괜찮은 것 같습니다! 또한 DTO 변환은 지금처럼 Converter로 하여, DTO는 단순 요청/응답 구조로만 사용하고 도메인 로직을 알지 못하게 책임을 분리하는 게 맞다고 생각합니다.

다음 이슈로 이메일 인증 로직을 언급하셨는데, 제 생각에는 소셜 로그인도 JWT 발급으로 이어져야해서 JWT 발급 및 로그인 로직을 먼저 이슈로 두는 게 좋을 것 같아요!

[kingmingyu]

고생하셨습니다! 코드 확인해봤는데 @Vaild 검사 실패까지 꼼꼼하게 구현해주신 것 같습니다! 저도 converter로 변환하는게 좋은 것 같습니다!