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

fix: 공연 전체 목록 조회 기능 수정 #136

Merged
merged 3 commits into from
Aug 27, 2024

Conversation

GaBaljaintheroom
Copy link
Contributor

😋 작업한 내용

  • 공연 목록 조회 시 인기순, 임박순 기준 정렬 후 무한 스크롤을 적용했습니다.
  • CursorValue를 front에 넘겨주는 방식으로 해보면 서브쿼리를 줄일 수 있는 장점으로 이전에는 그렇게 정하였는데요, 지금 같은 경우는 인기순, 임박순의 cursorValue 타입이 다르니 서브쿼리로 cursorValue를 찾아 해결 했습니다. 찬기님 말씀대로 정렬 하는 무한 스크롤 방식을 다시 한번 생각해봐야겠어요 ㅠㅡㅠ. 걍 서브쿼리가 맞는것 같기도 하고..? ㅎ.ㅎ 이건 이후에 논의해봐용
  • 응답 필드를 필요한 데이터에 맞게 수정했습니다.

🙏 PR Point

👍 관련 이슈


@GaBaljaintheroom GaBaljaintheroom added feat add feature fix fix feature labels Aug 27, 2024
@GaBaljaintheroom GaBaljaintheroom self-assigned this Aug 27, 2024
@GaBaljaintheroom GaBaljaintheroom merged commit 7aff7b5 into develop Aug 27, 2024
1 check passed
@GaBaljaintheroom GaBaljaintheroom deleted the fix/show_list branch August 28, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat add feature fix fix feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 전체 공연 목록 조회 API 개선
1 participant