좋아요 기능의 응답 코드를 통일 #692
Merged
좋아요 기능의 응답 코드를 통일 #692
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
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.
이전의 좋아요 기능 응답 코드는 다음과 같았습니다.
200 응답은 좋아요를 하나의 동작으로 본 경우, 204 응답은 좋아요를 하나의 자원으로 본 경우라고 생각해요.
좋아요를 하나의 동작으로 본다면 200 / 200 응답이, 하나의 자원으로 본다면 201 / 204 응답이 적절할 것 같습니다.
이에 대한 논의는 아마 각자의 선호도가 달라 통일시키기 어려울 것으로 보입니다.
또한, 옳고 그름의 문제가 아닌지라 어떤 선택이든 납득이 가능하기도 해요.
그래서 헤인의 "그렇다면 예외 처리 하기 편하게 200 으로 던져줘도 될까?" 라는 요청을 고려해 200 / 200 응답으로 수정합니다.