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

refactor: 카테고리 조회 상품 상세 조회 api swagger 반영 #81

Conversation

Combi153
Copy link
Contributor

📌 관련 이슈

📁 작업 설명

  • 상품 상세 조회, 카테고리 조건 상품 조회 API 에 대한 Swagger 설정이 누락되어 추가했습니다.
  • 카테고리 조건 상품 조회 API는 LoginMemberOrGuest 도 추가했습니다.
  • Swagger 테스트하는 김에 더미데이터 생성 로직을 변경했습니다. 나름 데이터 정합성을 맞추려고 노력했습니다!

Copy link

Test Results

 28 files  ±0   28 suites  ±0   9s ⏱️ ±0s
134 tests +1  134 ✅ +1  0 💤 ±0  0 ❌ ±0 
179 runs  +2  179 ✅ +2  0 💤 ±0  0 ❌ ±0 

Results for commit 59047a0. ± Comparison against base commit 2315865.

canDeliverSafely = true,
canDeliverCommonly = true,
canPickUp = true,
description = "https://www.goldmoonaqua.com/web/upload/NNEditor/20221226/copy-1672038777-guppy_EC958CEBB984EB85B8ED9280EBA088EB939C_02.png",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

수족관들은 상품 상세보기에 대한 내용을 대부분 image 파일로 처리하더라구요! 그래서 더미데이터에 저렇게 넣어두었습니다.

https://goldmoonaqua.com/product/%EC%95%8C%EB%B9%84%EB%85%B8-%ED%92%80-%EB%A0%88%EB%93%9C-%EA%B5%AC%ED%94%BC-1%EC%8C%8D/265/category/1/display/3/

Copy link
Contributor

@hgo641 hgo641 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

더미데이터 좋은데요?? 👍
고생하셨습니다! 깔끔해서 리뷰달게 없네요!ㅎㅎ
어프루브하겠습니다!

@@ -27,6 +29,7 @@ class ProductController(

@Operation(summary = "상품 상세 조회 API", description = "상품의 상세 정보를 조회합니다")
@ApiResponse(responseCode = "200", description = "상품 상세 조회 성공")
@SecurityRequirement(name = ACCESS_TOKEN_SECURITY_SCHEME_KEY)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

인증이 선택적으로 필요한 api에도 이 어노테이션 붙이면 되는건가요??

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 맞습니다! 저 친구가 있으면 동일하게 Authorization header 에서 값을 가져와요!

@Combi153 Combi153 merged commit f1b8624 into develop Feb 15, 2024
2 checks passed
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.

refactor: 카테고리 조회, 상품 상세 조회 API Swagger 반영
3 participants