-
Notifications
You must be signed in to change notification settings - Fork 0
[FIX] 유저/소소 피드 조회 관련 API 수정 #357
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
Conversation
GiJungPark
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FeedGetOption과 SortCriteria 클래스를 Controller에서 바로 핸들링 하는 부분이 인상적이었습니다.
저의 경우에는 DTO에서 String으로 Blank처리 이후 서비스 로직에서 도메인 객체 생성하는 시점에서 핸들링을 주로 해왔었는데, 이런 방법도 있다는걸 배울 수 있었던 것 같아요!
고생하셨어요 :)
src/main/java/org/websoso/WSSServer/repository/FeedCustomRepositoryImpl.java
Show resolved
Hide resolved
EunjeongHeo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넘 수고하셨습니다~!
코멘트 몇 개 남겼으니 확인 부탁드려요!
src/main/java/org/websoso/WSSServer/repository/FeedCustomRepositoryImpl.java
Outdated
Show resolved
Hide resolved
sansan20535
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 반영 및 코멘트 추가했습니다! 감사합니다 : )
Related Issue
Key Changes
피드소소 사일로 관련 API 수정사항입니다.
[유저 피드 조회 API]
[소소 피드 조회 API]
요청 값 추가
응답 값 추가
[소소 피드 단건 조회 API]
[Enum Type Converter 추가]
[파라미터 값 null값 허용]
To Reviewers
References
유저 피드 조회 API
소소 피드 조회 API
소소 피드 단건 조회 API
기능명세서