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

스페이스 필터 조회 로직 수정 #290

Merged
merged 4 commits into from
Feb 2, 2024

Conversation

dudwns
Copy link
Contributor

@dudwns dudwns commented Feb 1, 2024

📑 이슈 번호

#289

🚧 구현 내용

  • 기존 오프셋 기반 페이지네이션(Offset-based Pagination)이 적용되었던 메인 페이지의 스페이스 리스트를 커서 기반 페이지네이션(Cursor-based Pagination) 방식으로 변경하였습니다.

  • 메인 페이지, 검색 페이지, 프로필 페이지에서 스페이스 검색 로직이 공통적으로 사용되고 있어서 메인 페이지만 변경할 수 있도록 useMainSpacesQuery MainSpaceList를 추가로 구현하였습니다.

스페이스 필터 조회 API 변경점

기존 API request 값

{
  pageNumber: number,
  pageSize: number,
  sort: string,
  filter: string,
}

변경 후 API request 값

{
  lastFavoriteCount: number,
  lastSpaceId: number,
  pageSize: number,
  sort: string,
  filter: string,
}

🚨 특이 사항

Cursor-based Pagination 관련 글인데 궁금하시면 한 번 읽어봐도 좋을 것 같습니다!
관련 블로그

@dudwns dudwns added ✨ Feature 기능 개발 📬 API 서버 API 통신 labels Feb 1, 2024
@dudwns dudwns self-assigned this Feb 1, 2024
@dudwns dudwns linked an issue Feb 1, 2024 that may be closed by this pull request
@dudwns dudwns merged commit 4e2cbcb into develop Feb 2, 2024
1 check passed
@dudwns dudwns deleted the feature/#289/spaces-get-api branch February 2, 2024 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📬 API 서버 API 통신 ✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

스페이스 필터 조회 로직 수정
1 participant