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

[Feat/115] 게시판 글 목록 조회 필터 기능 구현 #120

Merged
merged 3 commits into from
Aug 8, 2024

Conversation

hzee97
Copy link
Contributor

@hzee97 hzee97 commented Aug 7, 2024

🚀 개요

게시판 글 목록 조회 필터 기능 구현

🔍 변경사항

  • 게시판 글 목록 조회 필터 기능 구현

⏳ 작업 내용

  • 게시판 글 목록 조회 필터 기능 구현

📝 논의사항

@hzee97 hzee97 requested review from rimi3226 and Eunjin3395 August 7, 2024 18:15
@hzee97 hzee97 self-assigned this Aug 7, 2024
@hzee97 hzee97 linked an issue Aug 7, 2024 that may be closed by this pull request
1 task
@@ -68,6 +68,7 @@ public static class boardListResponseDTO{
List<Long> championList;
Double winRate;
LocalDateTime createdAt;
Boolean voice;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

회원이랑 매칭에서 마이크 유무를 mike로 사용중인데 혹시 voice 대신에 mike를 사용해주실 수 있으신가요..?ㅠㅠ

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

변경 완료하였습니다.

Comment on lines +122 to +125
if (mainPosition != null && mainPosition == 0) {
// 전체 포지션 선택 시 필터링에서 제외
mainPosition = null;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

어차피 전체 포지션 선택 시 필터링 조건이 없는거라면 mainPosition을 0으로 입력받아야하는 이유가 있을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

포지션 선택을 안하거나 전체 포지션(*)을 선택할 경우 필터링하지 않고 전체 목록이 보이게 구현하였습니다.

@Eunjin3395 Eunjin3395 merged commit b1bdfaa into develop Aug 8, 2024
@rimi3226 rimi3226 deleted the feat/115 branch August 8, 2024 15:00
@hzee97 hzee97 restored the feat/115 branch August 9, 2024 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ [Feat] 게시판 글 목록 조회 필터 기능 구현
3 participants