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.

Sign up for GitHub

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

Jump to bottom

Feat: 이번달 러닝 서머리 V2 API 추가 #300

Merged
merged 2 commits into from
Nov 8, 2024
Merged

Feat: 이번달 러닝 서머리 V2 API 추가 #300

merged 2 commits into from
Nov 8, 2024

Conversation

hee9841
Copy link
Collaborator

@hee9841 hee9841 commented Nov 7, 2024 •
edited
Loading

🔗 이슈 연결

🚀 구현한 API

  • GET api/v2/running-records/monthly-summary

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

  • 리팩터링 관련
    • V1, V2 공통 리스펀스 DTO RunningRecordMonthlySummaryResponse로 지정합니다.
    • 공통으로 필요하는 month, monthlyTotalMeter 값을 서비스단의 getMonthlyRunningSummery함수에서 리턴하도록 변경합니다.
    • V1에서 추가로 필요한 다음 레벨 값과 다음 레벨까지 남은 거리수를 구하기 위해 MemberService의 getMyProfile 메서드로 가져옵니다.
    • 컨트롤러에서 위의 두개의 값을 가공해 RunningRecordMonthlySummaryResponseV1으로 리턴하도록 변경합니다.
  • V2 추가
    • 현재 사용자가 달리고 있는 지구 한 바퀴 코스를 조회하고, 코스에 대한 퍼센티지 값들을 계산해서 리턴하는 getPercentageValues 메서드를 RunningRecordServiceV2.java에 추가합니다.

🔍 리뷰 요청/참고 사항

  • 패키지 위치나, DTO명은 임시로 지정했어요. 버전 관련해서 구조를 어떻게 변경할지 결정되어 리팩터링 작업 들어가면 그 때 다시 변경하도록 하겠습니다.

@hee9841
Refactor: 이번달 러닝 서머리 V2를 위한 코드 리팩터링(임시)
8660eec 
- V2, V1 공통 리스펀스 DTO RunningRecordMonthlySummaryResponse로 지정(이번달, 이번달 달린 거리)
- V1 리스펀스 추가
- 서비스 로직 변경
- 컨트롤러에서 값 가공후 V1으로 리턴
@hee9841 hee9841 added the refactor 코드 리팩토링 label Nov 7, 2024
@hee9841 hee9841 self-assigned this Nov 7, 2024
@hee9841 hee9841 requested a review from Jaewon-pro November 7, 2024 12:23
@hee9841
Feat: running-records/monthly-summary의 V2 추가
4f93079 
- 현재 달리고 있는 지구 한바퀴 코스에 대한 퍼센테이지 값 조회하는 서비스단구현(임시)
- 컨트롤러 생성
- ResponseDto V2 생성
@hee9841 hee9841 changed the title Refactor: 이번달 러닝 서머리 V2를 위한 코드 리팩터링(임시) Feat: 이번달 러닝 서머리 V2 API 추가 Nov 8, 2024
@hee9841 hee9841 added the feature 기능 추가 label Nov 8, 2024
@hee9841 hee9841 merged commit f134e93 into main Nov 8, 2024
1 check passed
@hee9841 hee9841 deleted the refactor/#294/running-monthly-V2 branch November 8, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jaewon-pro Jaewon-pro Awaiting requested review from Jaewon-pro

Assignees

@hee9841 hee9841

Labels
feature 기능 추가 refactor 코드 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

홈화면의 현재 달리고 있는 코스 그라데이션 그래프 작업
1 participant
@hee9841