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

설계한 ERD를 토대로 JPA 엔티티 클래스들을 추가 #25

Merged
merged 7 commits into from
Aug 1, 2024

Conversation

Jaewon-pro
Copy link
Member

@Jaewon-pro Jaewon-pro commented Jul 28, 2024

🔗 이슈 연결

🚀 구현한 API

  • X

💡 반영할 내용 및 변경 사항 요약

  • JPA 엔티티 클래스들을 추가합니다.

DND 11th - 2

🔍 리뷰 요청/참고 사항

  • erd 설계가 잘 되었는지 궁금합니다.
  • 나머지는 댓글로 달겠습니다!

Jaewon-pro and others added 5 commits July 26, 2024 20:05
* Feat: member profile 추가

* Feat: badge 엔티티 추가

* Feat: level 엔티티 추가
* Chore: PostGIS 의존성 추가

* Chore: PostGIS 관련 옵션 application.yml에 추가

* Feat: RunningRecord 엔티티 추가

* Feat: RunningEmoji 엔티티 추가

* Feat: RoutePoint 엔티티 추가

* Style: PostGIS 의존성 위치 수정

* Feat: RunningRecord, RunningEmoji 테이블 리뷰 사항 반영
@Jaewon-pro Jaewon-pro added the feature 기능 추가 label Jul 28, 2024
@miue22
Copy link

miue22 commented Jul 29, 2024

  1. 테이블 전반적으로 카멜 케이스와 스네이크 케이스가 혼용되는 것 같습니다. DB도 일관성을 위해 컨벤션을 정해서 사용하시는게 좋을 것 같아요~
  2. Level 테이블에 '레벨 이름'은 어떤 역할을 하는 데이터인가요?
  3. BadgeArchievement 테이블에 배지를 획득한 날짜 정보가 들어가면 좋을 것 같아요~
  4. Member 테이블은 앞으로 확장될 가능성이 높아보이는데요, 너무 비대해지기 전에 분리하는 것도 좋아보입니다. oauth처럼 유저의 의지와 무관하게 관리되는 정보(=회원)와 키, 몸무게처럼 유저가 수정할 수 있는 정보(=프로필) 정도로 나누는게 어떨까요?

@Jaewon-pro
Copy link
Member Author

Level 테이블에 '레벨 이름'은 어떤 역할을 하는 데이터인가요?

레벨 테이블에 문자열 이름(레벨 1, 또는 브론즈..)을 넣도록 했는데, 생각해보니 문자열이 아니라 레벨 1, 2, .. 으로 숫자만 있을 것 같아요.
그래서 MemberLevel 테이블에서 사용자 레벨은 Level 테이블 id 값을 사용하도록 하면 좋을까요?

Member 테이블은 앞으로 확장될 가능성이 높아보이는데요, 너무 비대해지기 전에 분리하는 것도 좋아보입니다. oauth처럼 유저의 의지와 무관하게 관리되는 정보(=회원)와 키, 몸무게처럼 유저가 수정할 수 있는 정보(=프로필) 정도로 나누는게 어떨까요?

oauth 정보는 유저가 업데이트하는 경우가 없다고 생각해서, 소셜 로그인 정보를 member 테이블에서 분리하도록 하겠습니다!

@Jaewon-pro Jaewon-pro merged commit cf158ad into main Aug 1, 2024
1 check passed
@Jaewon-pro Jaewon-pro deleted the develop/#14/table branch August 1, 2024 14:03
@Jaewon-pro Jaewon-pro changed the title 설계한 ERD롤 토대로 JPA 엔티티 클래스들을 추가 설계한 ERD를 토대로 JPA 엔티티 클래스들을 추가 Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ERD 구성
3 participants