Feature/세미나 출석 정보 조회시 기수와 이름 정렬을 추가한다.#417
Merged
Conversation
- 기수 순으로 정렬한다. - 동일 기수 내라면 이름 순으로 정렬한다. #394
lcqff
added
🚀 feature
JJimini
approved these changes
Mar 1, 2024
Collaborator
JJimini
left a comment
JJimini
left a comment
There was a problem hiding this comment.
회의 시간에 말씀하셨던 거군요! 확인했습니다~!
stopmin
approved these changes
Mar 4, 2024
Collaborator
stopmin
left a comment
stopmin
left a comment
There was a problem hiding this comment.
수고하셨습니다!
정렬하는 것은 비지니스적인 로직이 아니라, 백엔드 단에서 프론트에게 보내는 과정에서 필요한 부분이라 생각해서 Controller단에서 해도 상관없다고 생각합니다!
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
🔥 Related Issue
close: #394
📝 Description
기존의 세미나 출석 정보 조회는 생성된 회원 순으로 정렬됩니다.
세미나 출석 정보 조회를 기수 순으로 정렬하고, 동일 기수 내라면 이름 순으로 정렬하는 코드를 추가했습니다.
⭐️ Review Request
정렬 Controller 코드 단에서 하는 걸로 괜찮을까요?
테스트 코드를 따로 작성 안하고 기존 테스트 코드만 살짝 수정했습니다. 기수 순으로도 잘 정렬 되는지 테스트 추가하고 싶었는데 Generation을 설정하려면 엔티티 코드를 건드려야 할거 같아서 일단 뺐습니다... (기수순으로도 정렬 되는거 확인은 했습니다!)
추가적인 테스트 코드 필요할 거 같으면 말씀해주세요~!