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

[feature/store-get-detail] 주변 식당 리뷰 조회 기능을 개발 #80

Merged
merged 5 commits into from
Jan 26, 2024

Conversation

Qbeom0925
Copy link
Member

작업 내용

주변 식당 리뷰 조회 기능을 개발하였습니다.
새로운 API 엔드포인트 /api/v1/stores/{storeIdx}/reviews를 추가하였습니다.
StoreService에서 주변 식당 리뷰를 조회하는 메서드를 구현하였습니다.
새로운 응답 DTO 클래스 StoreGetReviewRes를 추가하였습니다.
관련된 테스트 코드를 작성하였습니다.

관련 이슈

#79

작업 확인 방법

API 엔드포인트 /api/v1/stores/{storeIdx}/reviews를 호출하여 주변 식당 리뷰를 조회합니다.
테스트 코드를 실행하여 주변 식당 리뷰 조회가 올바르게 동작하는지 확인합니다.

추가 정보 (선택 사항)

변경된 코드는 다음과 같습니다.
StoreController: 식당 리뷰 조회 엔드포인트 추가
StoreService: 주변 식당 리뷰 조회 메서드 추가
StoreGetReviewRes: 새로운 응답 DTO 클래스
StoreMapper: 주변 식당 리뷰 조회를 위한 SQL 쿼리 추가
StoreServiceImplTest: 주변 식당 리뷰 조회 테스트 추가

@Qbeom0925 Qbeom0925 self-assigned this Jan 24, 2024
Copy link

sonarcloud bot commented Jan 25, 2024

Copy link
Member

@dldmsql dldmsql left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Comment on lines -31 to +50
스웨거 한계로 인해 Response에 공통 response의 data에 대한 정의가 되어있습니다. <br>""");
스웨거 한계로 인해 Response에 공통 response의 data에 대한 정의가 되어있습니다. <br>
<details>
<summary> ERROR LIST </summary>
<table>
<thead>
<tr>
<th>ERROR CODE</th>
<th>STATUS</th>
<th>ERROR MESSAGE</th>
</tr>
</thead>
<tbody>
"""
+ getErrorList()
+ """
</tbody>
<table>
</details>
""");
Copy link
Member

Choose a reason for hiding this comment

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

이것이... 에러코드!?!!인가요.
당신은 .. 혹시 천재..?!@

@Qbeom0925 Qbeom0925 merged commit ba26d8e into develop Jan 26, 2024
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.

2 participants