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

게임 도메인 양방향 연관관계 제거 #281

Merged
merged 2 commits into from
Feb 2, 2024

Conversation

charlesuu
Copy link
Collaborator

@charlesuu charlesuu commented Feb 2, 2024

👨‍💻 작업 사항

📑 PR 개요

게임 도메인 관련 양방향 연관관계를 제거한다.


✅ 작업 목록

  • Embedded 애너테이션 붙어있는 VO 제거
  • 양방향 연관관계 관련 메서드를 모두 쿼리 메서드로 변경
  • findByXXX에 대한 예외처리를 JpaRepository 인터페이스 내에서 디폴트 메서드로 변경

Prefix

PR 코멘트를 작성할 때 항상 Prefix를 붙여주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@charlesuu charlesuu added the 리팩토링 [refactor] 실행 결과는 동일하고, 코드 구조만 변경한다. label Feb 2, 2024
@charlesuu charlesuu self-assigned this Feb 2, 2024
@kylekim2123 kylekim2123 changed the base branch from dev to refactoring February 2, 2024 06:38
@charlesuu charlesuu merged commit f5fcb03 into refactoring Feb 2, 2024
@charlesuu charlesuu deleted the game-refactoring branch February 2, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
리팩토링 [refactor] 실행 결과는 동일하고, 코드 구조만 변경한다.
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

2 participants