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: 챌린지 및 목표 응답 DTO 통합 #326

Merged
merged 10 commits into from
Dec 26, 2024

Conversation

hee9841
Copy link
Collaborator

@hee9841 hee9841 commented Dec 26, 2024

🔗 이슈 연결

🚀 구현한 API

  • X

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

  • 러닝 결과의 챌린지 및 목표 달성 관련해서 응답 형식을 통합했습니다.
  • percentage값은 사용자가 퍼센티이지를 나타내지 못하는 기록을 조회 할 경우를 null를 리턴하도록 구성했습니다.
  • RunningRecordServcie.java의 calChallengeAchievementPercentage 는 이후 러닝 기록 조회 시 재 사용할 예정입니다.

🔍 리뷰 요청/참고 사항

  • 오늘 안에 챌린지 및 목표 DTO 통합이 완료 되어야 되어서 러닝 결과 조회 V2 API도 같은 PR에 올리게 되었습니다.
  • 코드 리뷰하기에 시간이 부족하면, 오늘 먼저 머지 진행하고, 금요일 스크럼에서 이야기 나누는 건 어떠세요?

@hee9841 hee9841 added the feature 기능 추가 label Dec 26, 2024
@hee9841 hee9841 self-assigned this Dec 26, 2024
@hee9841 hee9841 requested a review from Jaewon-pro December 26, 2024 07:54
@hee9841 hee9841 changed the title Feat: 챌린지 및 목표 응답 DTO 통합(러닝 결과 추가) Feat: 챌린지 및 목표 응답 DTO 통합 Dec 26, 2024
Copy link
Member

@Jaewon-pro Jaewon-pro left a comment

Choose a reason for hiding this comment

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

내일 스크럼때 이야기 해보면 좋을 것 같아요! 수고하셨습니다 👍

@hee9841 hee9841 merged commit 027d91b into main Dec 26, 2024
1 check passed
@hee9841 hee9841 deleted the feature/#317/challeng-goal-dto-for-add branch December 28, 2024 05:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants