-
Notifications
You must be signed in to change notification settings - Fork 7
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
카테고리 생성 시 생성된 카테고리 값 반환 #485
카테고리 생성 시 생성된 카테고리 값 반환 #485
Conversation
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.
고생하셨어요 켬미~~!
ResponseEntity<CreateCategoryResponse> createCategory(MemberDto memberDto, | ||
CreateCategoryRequest createCategoryRequest | ||
); |
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.
급한 내용이라면 무시해도 좋을 듯 합니다!
반환값이 void -> CreateCategoryResopnse
로 바뀐 부분도 문서화가 되어 있으면 좋을 것 같네요!
ApiResponse
부분에 아래 코드가 추가되면 어떨까요?
content = {@Content(schema = @Schema(implementation = CreateCategoryResponse.class))}
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.
그런데 다시 다른 문서화들을 보니 해당 코드가 거의 다 없는 것 같기도 하고~~
그냥 나중에 한번에 추가하는 PR 이 생기는 게 더 편할 것 같기도 합니다 크크
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.
일부로 모든 성공 API에서 @Schema 제거 했습니다 ~
제거해도 반환하는 Schema로 자동 Swagger 설정이 됩니다 ~
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.
고생하셨습니다!
ResponseEntity<CreateCategoryResponse> createCategory(MemberDto memberDto, | ||
CreateCategoryRequest createCategoryRequest | ||
); |
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.
이제까지 켬미의 리팩토링 스타일을 보면 아래와 같이 수정되어야 할 것 같은데 맞을까요?
ResponseEntity<CreateCategoryResponse> createCategory(MemberDto memberDto, | |
CreateCategoryRequest createCategoryRequest | |
); | |
ResponseEntity<CreateCategoryResponse> createCategory( | |
MemberDto memberDto, | |
CreateCategoryRequest createCategoryRequest | |
); |
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.
👍
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.
켬미 수고했어요 🍀
변경 사항