Skip to content

[SCRUM-235] 내 리뷰 조회 API 구현 (#95)#97

Merged
ssggii merged 4 commits intodevelopfrom
SCRUM-235-내-리뷰-조회-API-구현
Aug 13, 2025

Hidden character warning

The head ref may contain hidden characters: "SCRUM-235-\ub0b4-\ub9ac\ubdf0-\uc870\ud68c-API-\uad6c\ud604"
Merged

[SCRUM-235] 내 리뷰 조회 API 구현 (#95)#97
ssggii merged 4 commits intodevelopfrom
SCRUM-235-내-리뷰-조회-API-구현

Conversation

@ssggii
Copy link
Contributor

@ssggii ssggii commented Aug 10, 2025

✏️ 연관 이슈

#95

⛳ 작업 내용

  • MyReviewInfoDto 생성
  • 내 리뷰 조회 레포지토리, 서비스, 컨트롤러 메서드 구현
  • 테스트 코드 구현
  • swagger 호출 테스트

💬리뷰 요구사항

.

📍 참고사항

.

ssggii added 2 commits August 10, 2025 18:07
마이페이지에서 사용할 내 리뷰 조회 API를 구현했습니다.

- MyReviewInfoDto 생성
- ReviewController.getMyReviews() 구현
- ReviewService.getMyReviews() 구현
- ReviewRepository.findReviewsWithDetailsByMemberId() 구현
내 리뷰 조회 API의 서비스 메서드에 대하여 테스트 코드를 구현했습니다.
테스트 케이스는 다음과 같습니다.

- 내 리뷰 조회 성공 (작성한 리뷰가 있을 때)
- 내 리뷰 조회 성공 (작성한 리뷰가 없을 때)
@ssggii ssggii added this to the #5-sprint milestone Aug 10, 2025
@ssggii ssggii requested a review from moonxxpower August 10, 2025 11:31
@ssggii ssggii self-assigned this Aug 10, 2025
@ssggii ssggii moved this to In Progress in Runchuck-러닝한다이 Aug 10, 2025
Copy link
Member

@moonxxpower moonxxpower left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 👍

현재 내 리뷰 조회 시 최신 등록일 순이 아닌거 같은데 혹시 기획에서 바뀐 부분이 있었을까요?
피그마 스토리보드 상에서 최신 등록일 순이라고 명시되어 있어 여쭈어봅니다!
스크린샷 2025-08-13 오전 6 23 25

@ssggii
Copy link
Contributor Author

ssggii commented Aug 13, 2025

여기도 제가 리뷰 정렬 조건이 있는 줄 몰랐네요😅 리뷰 최신순 정렬로 수정해놓겠습니다!
요구사항 꼼꼼히 체크해주셔서 감사합니다☺️

ssggii added 2 commits August 13, 2025 15:42
내 리뷰 조회 API의 엔드포인트를 /api/me로 시작하도록 수정했습니다.
내 리뷰 조회 시 리뷰들이 최신순으로 반환되도록 정렬 조건을 추가했습니다.
@ssggii ssggii merged commit f7f042b into develop Aug 13, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Runchuck-러닝한다이 Aug 13, 2025
@ssggii ssggii deleted the SCRUM-235-내-리뷰-조회-API-구현 branch August 13, 2025 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants