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

[#122] feat: 플레이그라운드 마이페이지 내 모임 영역 데이터 조회 api #123

Merged
merged 3 commits into from
Jan 27, 2024

Conversation

yeseul106
Copy link
Member

@yeseul106 yeseul106 commented Jan 24, 2024

👩‍💻 Contents

  • pagination 관련 파일 추가해서 api 개발 진행했습니다.

  • 플레이그라운드 마이페이지 내 모임 영역에 필요한 데이터 컬럼만 뽑아서 내려줄 수 있도록 구현했습니다.

    • id : 모임 id
    • isMeetingLeader : 해당 모임의 개설자인지
    • title : 모임 타이틀
    • imageUrl : 모임 썸네일 Url
    • category : 모임 카테고리
    • mStartDate : 모임 활동 시작 날짜
    • mEndDate : 모임 활동 종료 날짜
    • isActiveMeeting: 활동중인 모임인지 여부
image
  • 영우님과 코드 포맷팅이 다르게 설정되어있었나봅니다 ㅠㅠ 어떤걸로 세팅해두셨는지 알려주시면 제가 바꿔볼게요..!

📝 Review Note

pagination 관해서 한 가지 의견을 여쭤보고 싶은게 있는데요...!

지금 이 api의 서비스 로직의 경우, 내가 참여한 모임 리스트를 조회하다 보니 내가 개설한 모임내가 지원하고 참여한 모임 두 개를 합쳐서 내려줘야하는데요.

apply 테이블과 조인을 해서 meeting 데이터를 가져온 것과 meeting 테이블에서 userId를 통해 가져온 데이터를 합쳐서 내려줄려면 DB 조회 쿼리 로직이 두 개로 나눠져야한다고 판단했습니다.

그러다보니 DB 계층에서 페이지네이션이 되는게 아니라 서비스 계층에서 페이지네이션을 구현할 수 밖에 없다고 생각해서 일단 그렇게 구현해놓기는 했습니다.

혹시 제가 생각 못했던 방법이 있거나, 잘못 생각한 부분이 있는지 영우님 의견 들어보고 싶습니다 :)

📣 Related Issue

@yeseul106 yeseul106 added the 🎁 feature 새로운 기능 label Jan 24, 2024
@yeseul106 yeseul106 self-assigned this Jan 24, 2024
@rdd9223
Copy link
Contributor

rdd9223 commented Jan 26, 2024

확인했습니다!

영우님과 코드 포맷팅이 다르게 설정되어있었나봅니다 ㅠㅠ 어떤걸로 세팅해두셨는지 알려주시면 제가 바꿔볼게요..!

이건 .editorconfig 파일로 설정하면 vscode, intelliJ 둘다 설정 가능할 것 같은데 이것도 해야겠네요

지금 이 api의 서비스 로직의 경우, 내가 참여한 모임 리스트를 조회하다 보니 내가 개설한 모임과 내가 지원하고 참여한 모임 두 개를 합쳐서 내려줘야하는데요.

이 경우는 조회를 다음과 같이하면 어떨까 하는데... 의견 부탁드립니다. 여기서 사용자id는 orgId를 통해 DB에서 조회하는 일련의 validation을 거친 후에 자연스럽게 가지고 있을 거라고 기대하고 있어요.

select 미팅 from 미팅 left join 요청 where 미팅.userId = '사용자id' or 요청.userId = '사용자id' orderby ...  limit ... 

@yeseul106
Copy link
Member Author

due date 이슈로 일단 머지시킵니다 ! 나중에 영우님 리뷰 반영해서 리팩토링을 진행하는 방식으로 하겠습니다 !

@yeseul106 yeseul106 merged commit 0081c1d into develop Jan 27, 2024
@mikekks mikekks deleted the feature/#122 branch August 4, 2024 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎁 feature 새로운 기능 size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

플레이그라운드 프로필 내 모임 노출을 위한 모임 정보 조회 기능 구현
2 participants