[#122] feat: 플레이그라운드 마이페이지 내 모임 영역 데이터 조회 api #123
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
👩💻 Contents
pagination 관련 파일 추가해서 api 개발 진행했습니다.
플레이그라운드 마이페이지 내 모임 영역에 필요한 데이터 컬럼만 뽑아서 내려줄 수 있도록 구현했습니다.
📝 Review Note
pagination 관해서 한 가지 의견을 여쭤보고 싶은게 있는데요...!
지금 이 api의 서비스 로직의 경우, 내가 참여한 모임 리스트를 조회하다 보니
내가 개설한 모임
과내가 지원하고 참여한 모임
두 개를 합쳐서 내려줘야하는데요.apply 테이블과 조인을 해서 meeting 데이터를 가져온 것과 meeting 테이블에서 userId를 통해 가져온 데이터를 합쳐서 내려줄려면 DB 조회 쿼리 로직이 두 개로 나눠져야한다고 판단했습니다.
그러다보니 DB 계층에서 페이지네이션이 되는게 아니라 서비스 계층에서 페이지네이션을 구현할 수 밖에 없다고 생각해서 일단 그렇게 구현해놓기는 했습니다.
혹시 제가 생각 못했던 방법이 있거나, 잘못 생각한 부분이 있는지 영우님 의견 들어보고 싶습니다 :)
📣 Related Issue