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] 매니저가 생성한 리그와 리그의 경기중 게임 전체 조회 기능 #230

Merged
merged 5 commits into from
Sep 26, 2024

Conversation

Zena0128
Copy link
Contributor

🌍 이슈 번호

📝 구현 내용

  • 매니저가 생성한 리그와 리그의 경기중 게임 전체 조회 기능 추가

🍀 확인해야 할 부분

  • 이미 구현되었던 흔적?이 있어 해당 api 일부 수정해서 구현([FEAT] 매니저가 생성한 리그 전체 조회 #194)
    • [FEAT] 매니저가 생성한 리그 전체 조회 #194 가 머지 승인은 됐지만 머지는 되지 않은 것으로 알고 있는데 왜때문인지 이미 메인 브랜치에 코드가 남아있어서.. 일단 그거 활용해서 구현하긴 했는데 왜 코드가 있는 것인지,,?,,? 혹시 다른 곳에서 이미 사용되고 있는 api인지?!?
  • fetch join은 이너조인 방식으로 구현돼 리그에 리그팀이 아직 없는 경우 해당 리그가 조회되지 않는 문제 발견 => left fetch join으로 변경

Copy link
Contributor

@Jin409 Jin409 left a comment

Choose a reason for hiding this comment

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

수고하셨습니당

@@ -14,7 +14,7 @@ public interface LeagueQueryRepository extends Repository<League, Long>, LeagueQ

@Query(
"SELECT l FROM League l "
+ "JOIN FETCH l.leagueTeams "
+ "LEFT JOIN FETCH l.leagueTeams "
Copy link
Contributor

Choose a reason for hiding this comment

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

헉 이런 치명적인 버그가 있었다니..
굳굳 감사합니당 👍🏻👍🏻

Copy link
Contributor

@ldk980130 ldk980130 left a comment

Choose a reason for hiding this comment

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

고칠건 특별히 없어 보이네요! 수고하셨습니다~

@Zena0128 Zena0128 merged commit 5976ea7 into main Sep 26, 2024
1 check passed
@Zena0128 Zena0128 deleted the feat/#227-league branch September 26, 2024 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants