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] 엔티티 클래스 추가 #7

Merged
merged 7 commits into from
Jan 3, 2024
Merged

Conversation

jun02160
Copy link
Member

@jun02160 jun02160 commented Jan 2, 2024

📌 관련 이슈

closed #6

✨ 어떤 이유로 변경된 내용인지

변경 사항

  1. User 엔티티의 socialType 필드명

    부모/자녀를 구분하는 userType과 비슷해서 혼동의 여지가 있다고 판단 ⇒ socialPlatform으로 변경

  2. 부모/자녀를 구분하는 UserType Enum에 온보딩 이전까지 default로 지정될 타입을 NONE 으로 지정 → 필드 자체를 nullable로 열어두지 않기 위함

  3. 예외처리 부분에서 아직 수정되지 않은 부분을 발견하여 수정했습니다! (ErrorType이 BusinessException의 인자로 들어간 부분을 모두 ExceptionType으로 변경)

+) final 필드로만 이루어진 Enum 클래스에는 @AllArgsConstructor 대신 @RequiredArgsConstructor를 사용하도록 일괄 변경

최종 패키지 구조

엔티티 클래스로 생성한 테이블 목록

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

  • DB 설계 회의에서 결정된 사항에서 변경된 부분만 정리해보았습니다!!
  • 혹시 놓친 부분이 있다면 말씀해주세요 :)

@jun02160 jun02160 added 🔥PULL REQUEST PR 날릴 때 ☁️DATABASE 서버 DB 작업 🧹MODIFY 코드 수정 (기능의 변화가 있을 때) 🐈예준🐈 labels Jan 2, 2024
@jun02160 jun02160 self-assigned this Jan 2, 2024
Copy link
Member

@hyeyeonnnnn hyeyeonnnnn left a comment

Choose a reason for hiding this comment

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

모두 확인했습니다 👍
설계한 대로 잘 구현된 것 같습니다!!
수고하셨습니다 갓예준님~~~~~

@jun02160 jun02160 merged commit f9ddeac into develop Jan 3, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
☁️DATABASE 서버 DB 작업 🐈예준🐈 🧹MODIFY 코드 수정 (기능의 변화가 있을 때) 🔥PULL REQUEST PR 날릴 때
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Entity 클래스 설계
2 participants