Skip to content
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

Feat: API 공통 핸들러 추가 #16

Closed
wants to merge 2 commits into from
Closed

Conversation

Jaewon-pro
Copy link
Member

🔗 이슈 연결

🚀 구현한 API

  • X

💡 반영할 내용 및 변경 사항 요약

  • API 공통 형식 핸들러를 추가합니다.
  • 아직 iOS팀과 상의하진 않아서 필드는 추후 회의하고 변경하면 좋을 것 같아요

🔍 리뷰 요청/참고 사항

  • ResponseBodyAdvice<Object>를 사용했는데, Controller에서 String을 반환하면 에러가 발생해요. 그런데 찾아봐도 못 찾겠어서 일단 ApiSuccessResponse로 감싸지 않고 그대로 반환하도록 했습니다.

@Jaewon-pro Jaewon-pro self-assigned this Jul 22, 2024
@Jaewon-pro Jaewon-pro requested a review from hee9841 July 22, 2024 13:11
@Jaewon-pro
Copy link
Member Author

데일리 스크럼 결과로 나온 형식에 맞게 다시 작성후 올리겠습니다.

새롭게 합의된 형식은 이슈를 참고해주세요

@Jaewon-pro Jaewon-pro closed this Jul 23, 2024
@Jaewon-pro Jaewon-pro deleted the feature/#15/api-response branch July 23, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API 응답 성공/예외 발생 처리 핸들러 필요
1 participant