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

FEAT: 10주차 과제 #5

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

FEAT: 10주차 과제 #5

wants to merge 2 commits into from

Conversation

wu-seong
Copy link
Collaborator

작업 내용 👨🏻‍💻

  1. 내가 작성한 리뷰 목록 조회 API 구현
  • StoreResponseDTO에 ReviewPreviewDTO, ReviewPreviewListDTO 추가
  • StoreConverter에 reviewPreviewDTO, reviewPreviewListDTO 추가
  • UserRestController에 getReviewList 메서드 추가 및 구현
  • UserQueryServic에 getReviewList 추가 및 구현
  • ReviewRepository에 findAllByUser 추가
  1. 특정 가게의 미션 목록 조회 API 구현
  • MissionResponseDTO에 MissionPreviewDTO, MissionPreviewListDTO 추가
  • MissionConverter에 missionPreviewDTO, missionPreviewListDTO 추가
  • StoreController에 getMissionList 메서드 추가 및 구현
  • storeQueryServic에 getMissionList 추가 및 구현
  • MissionRepository에 findAllByStore 추가
  1. 내가 진행중인 미션 목록 조회 API 구현
  • UserRestController에 getMissionList 메서드 추가 및 구현
  • userQueryServic에 getMissionList 추가 및 구현
  • UserMissionRepository 생성 및 findByUser메서드 추가

To Reviewers 💬

  • 워크북 가이드와 다르게 MissionController, Converter등을 따로 만들어서 진행을 했는데 이부분을 합치는게 프로젝트 구조를 볼 때 가독성 측면에서는 좋은 것 같긴한데 Mission은 User와 Store 두 엔티티와 연관 되어있어서 둘 중에 하나에 넣는 것보다 분리시키는게 관심사의 분리 측면에서는 좋은 것 같아서 이 부분에 대한 의견이 궁금합니다..! (같은 맥락에서 Review를 Store 내부에 넣은것 역시)

Reference 🔬

@wu-seong wu-seong self-assigned this Dec 26, 2023
@wu-seong wu-seong added the 🚀 mission mission label Dec 29, 2023
@wu-seong wu-seong changed the title 10주차 과제 FEAT: 10주차 과제 Dec 29, 2023
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.

1 participant