[SCRUM-228] 즐길거리 전체 조회 API 구현 (#80)#100
Merged
moonxxpower merged 9 commits intodevelopfrom Aug 14, 2025
Hidden character warning
The head ref may contain hidden characters: "SCRUM-228-\uc990\uae38\uac70\ub9ac-\uc804\uccb4-\uc870\ud68c-API-\uad6c\ud604"
Merged
Conversation
특정 코스의 즐길거리 전체 정보를 조회하는 API를 구현했습니다. 장소의 이름, 설명, 이미지를 응답합니다.
즐길거리 전체 조회 API의 서비스 메서드들에 대하여 테스트 코드를 구현하고 테스트를 진행했습니다. (성공 2개, 실패 1개)
코스 요약 조회 API에 즐길거리 정보를 추가했습니다. 즐길거리 정보는 랜덤으로 3개 응답하며, 이름, 설명, 이미지를 포함합니다. 관련 테스트 코드에도 즐길거리 정보 검증을 추가했습니다.
코스 요약 조회 API에서 전체 리뷰 개수 조회 로직을 수정했습니다.
코스 요약 조회 시 최신순 리뷰 2개를 반환하도록 수정했습니다.
ssggii
reviewed
Aug 12, 2025
Contributor
ssggii
left a comment
ssggii
left a comment
There was a problem hiding this comment.
리뷰 완료했습니다🙌
코스 요약 조회 API에서 수정 필요한 부분 추가로 커밋했는데 확인 부탁드립니다!
src/main/java/com/server/running_handai/domain/spot/service/SpotService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/server/running_handai/domain/spot/service/SpotService.java
Show resolved
Hide resolved
src/main/java/com/server/running_handai/domain/spot/service/SpotService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/server/running_handai/domain/spot/repository/SpotRepository.java
Outdated
Show resolved
Hide resolved
fetch join을 사용하는 메서드임을 명시하고, 기본 JPA 메서드와 구분하기 위해 findByCourseId에서 findByCourseIdWithSpotImage로 JPA 메서드명을 변경했습니다.
getSpotDetails에서는 Course 객체가 필요하지 않고 Course 존재 여부만 확인하고 있으므로 Course 존재 여부를 확인할 때 findById가 아닌 existsById를 사용하도록 변경했습니다.
코드 가독성을 위해 SpotInfoDto, SpotDetailDto 생성 로직을 서비스로부터 분리했습니다.
Member
Author
|
코스 요약 조회 API에서 추가 커밋하신 부분 확인했습니다! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✏️ 연관 이슈
#80
⛳ 작업 내용
/courses/{courseId}/spots)💬리뷰 요구사항
📍 참고사항