Skip to content

Conversation

@Sihun23
Copy link
Collaborator

@Sihun23 Sihun23 commented Feb 3, 2025

📄 요약(Summary)

✍🏼 상세(More)

PR Desciption

변경 사항 설명
-FindAllMembersService와 FindMembersWithFilterService에서 반환 타입을 PageResponse로 변경

  • 컨트롤러에서 페이징 응답을 PageResponse로 감싸서 반환하도록 수정
  • FindMemberRequest 필드에 @NotNull 추가
  • MemberCustomReposiotoryImpl 필터 조건 수정 != null -> ! isBlank()

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #258

@Sihun23 Sihun23 added 🐛 bug 버그를 수정합니다 🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다. labels Feb 3, 2025
@Sihun23 Sihun23 self-assigned this Feb 3, 2025
@Sihun23 Sihun23 changed the title CLAP-243 refactor:PageResponse 사용 및 dto mapping 서비스단으로 위임 CLAP-243 회원조회 API 수정 Feb 3, 2025
@Sihun23 Sihun23 requested a review from parkjaehak February 4, 2025 00:22
@Sihun23 Sihun23 merged commit c6e1f8d into develop Feb 4, 2025
1 check failed
@Sihun23 Sihun23 deleted the CLAP-243 branch February 4, 2025 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug 버그를 수정합니다 🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-243 회원조회 API 수정

3 participants