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.

Sign up for GitHub

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

Jump to bottom

[fix/#119] 페이지네이션 미적용 이슈 수정 #120

Merged
merged 1 commit into from
Jun 24, 2024
Merged

[fix/#119] 페이지네이션 미적용 이슈 수정 #120

merged 1 commit into from
Jun 24, 2024

Conversation

gmlrude
Copy link
Contributor

@gmlrude gmlrude commented Jun 24, 2024

📢 기능 설명

  • JOIN FETCH와 Pageable을 함께 사용할 때 처리하지 못할 때가 있음 → JOIN FETCH 대신 @EntityGraph 사용
  • QueryDSL은 직접적으로 Page 객체를 반환하는 기능 제공 X → 메서드 분리하여 적용

연결된 issue

연결된 issue를 자동으로 닫기 위해 아래 {이슈넘버}를 입력해주세요.

close #119

✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 이슈넘버를 적었는가?

@gmlrude gmlrude self-assigned this Jun 24, 2024
@gmlrude gmlrude merged commit b7103e2 into develop Jun 24, 2024
1 of 2 checks passed
@github-actions GitHub Actions
Copy link

Unit Test Results

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 0111a58. ± Comparison against base commit 47dd14e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@gmlrude gmlrude

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[fix] 페이지네이션 수정
1 participant
@gmlrude