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/#20] USER_ACTIVITY_HISTORIES, USERS, USER_REGISTER_INFOS 엔티티 구현 #21

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

sung-silver
Copy link
Contributor

Related Issue 🚀

Work Description ✏️

  • 이전에 설계했던 다이어그램을 바탕으로 엔티티를 구현했습니다
    MAKERS_AUTHENTICATION

PR Point 📸

  • @EnableJpaAuditing 어디에 붙이는게 맞을까요? 지금 구현 방향보면 core-api에 모두 붙이는 것 같긴한데..ㅎㅎ
  • user_activity_histories와 users를 양방향으로 가져가야할 것도 같은게 toDomain 메서드를 user 엔티티에서 만들려고하니 activities를 만들어서 반환할수가 없더라고요.. 다른 분들의 의견이 궁금합니다!
  • 회원가입 시점에 MemberEntity를 생성할 것으로 예상했는데, isActive 필드만 따로 파라미터로 받아도 될까요? 아니면 default는 false로 두고 업데이트를 하는게 나을까요?
  • 엔티티 생성 시 builder 패턴을 쓸까요 아니면 그냥 생성자를 사용할까요? 또는 불변 객체를 사용해서 만든다는 무엇의 함수를 따로 정의하는게 나을지...
  • user_register_info도 생성자가 필요할까요? 이건 회원가입 전에 저희가 db에 넣는 정보라고 생각하여 일단은 따로 만들지 않았습니다..!
  • 이 밖에도 놓친 부분이 있다면 많은 리뷰 부탁드립니다! 퇴근 20분 전이라 급하게 날려요 ㅎ

@sung-silver sung-silver added the 🎁 feature 새로운 기능을 개발하거나 추가, 변경할 경우(spring boot 내의 기능 코드) label Oct 31, 2024
@sung-silver sung-silver self-assigned this Oct 31, 2024
Copy link

height bot commented Oct 31, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎁 feature 새로운 기능을 개발하거나 추가, 변경할 경우(spring boot 내의 기능 코드) size/L
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

[FEAT] USER_ACTIVITY_HISTORIES, USERS, USER_REGISTER_INFOS 엔티티 구현
1 participant