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: 나의 기록 조회 V2 작업(퍼센테이지 값 추가) #299

Merged
merged 2 commits into from
Nov 6, 2024

Conversation

hee9841
Copy link
Collaborator

@hee9841 hee9841 commented Nov 6, 2024

🔗 이슈 연결

🚀 구현한 API

  • GET /api/v2/members/profiles

💡 반영할 내용 및 변경 사항 요약

  • 응답 버전 별 공통으로 값을 가공할 수 있게 MyProfileResponse의 필드 값을 변경합니다.
  • MemberService의 getMyProfile메서드가 MyProfileResponse를 리턴하도록 합니다.
  • 기존의 MyProfileResponseMyProfileResponseV1으로 변경합니다.
  • MyProfileResponseV1에서 MyProfileResponse값을 변환할 수 있게 합니다.
  • 퍼센테이지 값을 추가한 MyProfileResponseV2를 생성하고 /api/v2/members/profiles api를 추가합니다.

🔍 리뷰 요청/참고 사항

  • 해당 구조를 어떻게 변경할지는 백엔드 스크럼 회의에서 정하고 도메인 별로 리팩터링 작업 들어가면 될 것 같아요!
  • 우선 임시로 구성했습니다.
  • 리팩터링 코드랑, 구현 코드랑 별도로 PR를 올려야했었는데, 해당 작업이 목요일까지 완료하기로 예정되어서 같이 올렸어요! 다음부터는 나눠서 올리겠습니다🙏

- MyProfileResponseV1를 추가
- 응답 형식 확인 테스트 코드는 컨트롤러 테스트에 추가 예정
@hee9841 hee9841 added feature 기능 추가 refactor 코드 리팩토링 labels Nov 6, 2024
@hee9841 hee9841 self-assigned this Nov 6, 2024
@hee9841 hee9841 requested a review from WonSteps November 6, 2024 12:31
Copy link
Member

@WonSteps WonSteps left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 👍

@@ -1,17 +1,12 @@
package com.dnd.runus.presentation.v1.member.dto.response;
Copy link
Member

Choose a reason for hiding this comment

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

나중에 service 계층에서는 presentation에 의존하지 않도록 따로 수정하면 좋을 것 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

#301 이슈에서 해당 작업을 진행합니다.

@hee9841 hee9841 merged commit a3c395f into main Nov 6, 2024
1 check passed
@hee9841 hee9841 deleted the featrue/#295/profile-me-percentage branch November 6, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 추가 refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

나의 기록의 레벨 그라데이션 작업
2 participants