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.

Sign up for GitHub

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

Jump to bottom

⭐️ API Response HTTP 에러코드 정의 및 데이터 래핑 #19

Merged
merged 6 commits into from
Jul 13, 2023
Merged

⭐️ API Response HTTP 에러코드 정의 및 데이터 래핑 #19

merged 6 commits into from
Jul 13, 2023

Conversation

w8385
Copy link
Member

@w8385 w8385 commented Jul 13, 2023

업데이트 유형

  • Hot Fix
  • Release
  • Develop
  • Others

업데이트 개요

HTTP 에러 코드와 응답시 반환되는 데이터 래핑

업데이트 요약

  • API Response
    • HTTP 에러코드 정의
    • 각 요청시 예상되는 응답 스키마 정의
    • 응답 형식 정의 및 데이터 래핑

해결한 문제

  • docs: swagger #16
    에서 문제점으로 생각되었던 데이터 래핑 문제 해결

미해결 문제

  • 현재 레퍼런스로 남아있는 API들 정의 필요
  • Agora 과외 방 만드는 기능 필요하다고 생각됨

테스트

  • 유닛 테스트
  • 빌드 테스트 (통합 테스트)
  • 기타 유효성 테스트

수정 사항 진단

  • 코드가 이 프로젝트의 스타일 지침을 따릅니다.
  • 코드에 대한 자체 리뷰를 수행했습니다.
  • 변경 내역에 대해 주석을 작성했습니다.
  • 해당 PR에 대한 문서를 변경했습니다.
  • 기능, 수정 사항, 이슈에 대한 테스트 코드를 추가했습니다.
  • 변경 내용이 로컬 개발환경에서의 테스트를 통과했습니다.

@w8385 w8385 merged commit 3c40c47 into dev Jul 13, 2023
@w8385 w8385 deleted the feat/response branch July 13, 2023 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@w8385