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

[#104] 유저 개인 평균 행복지수 API 구축 #110

Merged
merged 10 commits into from
May 30, 2024
Merged

[#104] 유저 개인 평균 행복지수 API 구축 #110

merged 10 commits into from
May 30, 2024

Conversation

KkomSang
Copy link
Collaborator

#️⃣ 연관된 이슈

Resolves #104

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요 (이미지 첨부 가능)

리포트에 포함되는 유저 개인의 전체, 연간, 월간 평균 행복지수와 그에 따른 수준을 조회할 수 있습니다.

  • /api/report/all/happiness
{
  "success": true,
  "code": 0,
  "message": "유저 개인의 전체기간 평균 행복지수를 성공적으로 조회했습니다.",
  "data": {
    "happiness": 3.74,
    "level": "보통",
    "emoji": "😐"
  }
}
  • /api/report/year/happiness
{
  "success": true,
  "code": 0,
  "message": "유저 개인의 연간 평균 행복지수를 성공적으로 조회했습니다.",
  "data": {
    "happiness": 6.84,
    "level": "매우 높음",
    "emoji": "😄"
  }
}
  • /api/report/month/happiness
{
  "success": true,
  "code": 0,
  "message": "유저 개인의 월간 평균 행복지수를 성공적으로 조회했습니다.",
  "data": {
    "happiness": 4.6,
    "level": "보통",
    "emoji": "😐"
  }
}

변경사항

  • 평균 행복지수가 행복 트렌드뿐만 아니라 리포트에도 들어가므로 TrendHappinessResponseDto의 이름을 AverageHappinessResponseDto로 변경했습니다.
  • 위와 마찬가지 이유로 Trendconverter를 삭제하고 ReportConverter로 통합했습니다.
  • 위와 마찬가지 이유로 TrendHappinessService의 이름을 AverageHappinessService로 변경했습니다.
  • 저번에 말씀해주신 내용에 따라 HappinessLevel enum에 emoji를 추가하는 것이 더 나을 것이라고 생각하여 수정했습니다.
  • ReportGraphService에 삭제하지 않은 디버깅 코드를 발견하여 이를 삭제했습니다.

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

브랜치가,,,feat로 했어야 하는데, fix로 해부렀네요...하하

✅ Check List

  • PR 제목을 커밋 규칙에 맞게 작성했는가?
  • PR에 해당되는 Issue를 연결했는가?
  • 적절한 라벨을 설정했는가?
  • 작업한 사람을 모두 Assign했는가?

@KkomSang KkomSang added feat 새로운 기능을 추가합니다 fix 기능 및 버그를 수정합니다 labels May 29, 2024
@KkomSang KkomSang self-assigned this May 29, 2024
@yel-m yel-m self-requested a review May 29, 2024 15:23
@yel-m
Copy link
Member

yel-m commented May 29, 2024

코드 모두 확인했습니다!!
넘무나.. 깔꼼한 코드임니다
웹에서 해결하기 어려운 충돌이라고 떠서, 로컬에서 develop 브랜치와 병합한 다음 merge conflict를 해결해야 할 것 같습니다! 저랑 내일 같이 해봐융가리!!

@yel-m yel-m merged commit be6319c into develop May 30, 2024
1 check passed
@yel-m yel-m deleted the fix/#104 branch May 30, 2024 02:17
@yel-m yel-m mentioned this pull request May 30, 2024
4 tasks
KkomSang added a commit that referenced this pull request May 30, 2024
yel-m added a commit that referenced this pull request May 30, 2024
yel-m added a commit that referenced this pull request May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능을 추가합니다 fix 기능 및 버그를 수정합니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants