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
supports 메서드가 true 일때만, 해당 Advice가 동작 1. 컨버터 타입이 MappingJackson2HttpMessageConverter 이 2. 파라미터타입이 ResponseEntity가 아니고 3. 파라미터타입이 ProblemDetail가 아닐때
- HttpStatus를 가져오는 기능 - body 가 null이면 No content 를 body로 갖는 APIResponse 응답 - body 가 이미 APIResponse이면 바로 응답 - HttpStatus와 body로 APIResponse 객체를 응답
- APIResponseAdvice 가 APIResponse 객체를 생성해서 반환해야 함
- 컨트롤러는 responseDTO를 반환 - 응답 공통 처리는 Advice에서 수행 - @ResponseStatus 로 상태 코드 명시
- 컨트롤러는 responseDTO를 반환 - 응답 공통 처리는 Advice에서 수행 - @ResponseStatus 로 상태 코드 명시
- 컨트롤러는 responseDTO를 반환 - 응답 공통 처리는 Advice에서 수행 - @ResponseStatus 로 상태 코드 명시
Dr-KoKo
reviewed
Aug 14, 2024
8540ad5 반영 완료했습니다! |
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.
💡 다음 이슈를 해결했어요.
Issue Link - #32
APIResponseAdvice에서 컨트롤러의 responseDTO 를 APIResponse 로 응답하도록 공통 로직을 처리supports 메서드에서 해당 어드바이스가 처리해도 되는지 확인
MappingJackson2HttpMessageConverter컨버터가 아니면 무시beforeBodyWrite 에서 공통 로직을 수행
🔔 notice:
💡 이슈를 처리하면서 추가된 코드가 있어요.
기존 API 컨트롤러의 반환 타입 변경 및 @httpstatus 명시
[refactor] 구매자 회원가입 컨트롤러 반환타입 변경
[refactor] 계좌 충전 컨트롤러 반환타입 변경
[refactor] 점주 회원가입 컨트롤러 반환타입 변경
✅ 셀프 체크리스트