[Feature] 이동봉사자, 모집자 - 모집자 프로필 공고 조회 API 수정 #172
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 tasks
hojeong2747
approved these changes
May 4, 2024
| return ResponseEntity.ok(response); | ||
| } | ||
|
|
||
| @Operation(summary = "중개자 - 중개 프로필 - 이동봉사 공고 목록 조회", description = "중개 프로필에서 이동봉사 공고 목록을 조회합니다.", |
There was a problem hiding this comment.
이렇게 구분되어 써주는 거 좋은 것 같네요. 봉사자와 모집자가 더 잘 분리되는!
| new OrderSpecifier(Order.ASC, post.endDate), | ||
| new OrderSpecifier(Order.DESC, post.createdDate)} | ||
| : defaultOrder; | ||
| } |
There was a problem hiding this comment.
queryDsl에서 정렬할 때 쓰는 orderSpecifier 정확하게 알고 써본 적이 없어서 궁금했었는데 코드를 보니 감이 조금 잡히는 것 같아요. entity 필드 기준으로 정렬해야 하는 기능이 있을 때 저도 한 번 써보겠습니다!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
💡 연관된 이슈
close #171
📝 작업 내용
이동봉사자, 모집자 - 모집자 프로필 공고 조회 API 수정
💬 리뷰 요구 사항