Skip to content

[Refactor] 조회 API Swagger 쿼리 파라미터 표시 반영 - #288

Merged
yeonie0309 merged 2 commits into
developfrom
refactor/268-swagger-query-parameter
Aug 7, 2026
Merged

[Refactor] 조회 API Swagger 쿼리 파라미터 표시 반영#288
yeonie0309 merged 2 commits into
developfrom
refactor/268-swagger-query-parameter

Conversation

@yeonie0309

@yeonie0309 yeonie0309 commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

📌 PR 설명

  • GET /api/v1/reviews 요청 DTO에 @ParameterObject 추가
  • GET /api/v1/courses/{courseId}/reviews 요청 DTO에 @ParameterObject 추가
  • GET /api/v1/travel-records 요청 DTO에 @ParameterObject 추가
  • Swagger에서 목록 조회 요청 DTO가 query parameter로 표시되도록 수정

🔗 연관된 이슈

Resolves: #268

📝 리뷰 요구사항

  • Swagger에서 GET /api/v1/reviews 요청값이 request body가 아닌 query parameter로 표시되는지 확인 부탁드립니다.
  • Swagger에서 GET /api/v1/courses/{courseId}/reviews 요청값이 request body가 아닌 query parameter로 표시되는지 확인 부탁드립니다.
  • Swagger에서 GET /api/v1/travel-records 요청값이 request body가 아닌 query parameter로 표시되는지 확인 부탁드립니다.
  • 기존 API URL, 응답 구조, 실제 요청 방식이 유지되는지 확인 부탁드립니다.

💻 테스트 결과

  • API 테스트 완료
  • 정상 동작 확인
image image image

@yeonie0309 yeonie0309 self-assigned this Aug 6, 2026
@Seoyeon-park0103

Copy link
Copy Markdown
Collaborator

요청해 주신 세 API 모두 Swagger에서 request body가 아닌 query parameter로 정상 표시되는 것을 확인했습니다. 기존 API URL과 응답 구조 및 실제 요청 방식도 그대로 유지되어 승인하겠습니다! 수고하셨습니다~!!

@yeonie0309
yeonie0309 merged commit 2ac8663 into develop Aug 7, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 조회 API Swagger 쿼리 파라미터 표시 반영

2 participants