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

[TNT-253] fix: 회원 탈퇴 시 식단 데이터 삭제, 기타 수정 #64

Merged
merged 7 commits into from
Feb 14, 2025

Conversation

fakerdeft
Copy link
Contributor

📋 Checklist

  • 🔀 PR 제목의 형식을 잘 작성했나요? (e.g. [APP2-77] feat: 회원 인증 Filter 구현)
  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드에 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?

🎟️ Issue

✅ Tasks

  • [회원] 회원 탈퇴 API

    • 트레이니 회원 탈퇴 시 식단 데이터도 soft delete 하도록 수정했습니다.
  • [트레이니] 홈 특정 날짜 기록 조회 API

    • PT 수업 없는 날은 응답에서 {"ptInfo": null} 로 나오도록 수정했습니다.
  • swagger에서 키, 몸무게 nullable = true 로 수정했습니다.

🙋🏻 More

  • 참고 내용

@fakerdeft fakerdeft added 🐞 bugfix 버그 수정 ✅ test 테스트 코드 수정 ♻ refactor 기존 코드 리팩토링 labels Feb 14, 2025
@fakerdeft fakerdeft requested a review from ymkim97 February 14, 2025 00:24
@fakerdeft fakerdeft self-assigned this Feb 14, 2025
Copy link
Contributor

@ymkim97 ymkim97 left a comment

Choose a reason for hiding this comment

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

수정 고생하셨습니다!
PtService 쪽 하나만 확인 부탁드려요~!

@@ -377,4 +376,13 @@ private int validateAndGetNextSession(PtTrainerTrainee ptTrainerTrainee, LocalDa

return ptTrainerTrainee.getFinishedPtCount() + 1 + temp;
}

private GetTraineeDailyRecordsResponse.PtInfo toPtInfo(TraineeProjection.PtInfoDto dto) {
Copy link
Contributor

Choose a reason for hiding this comment

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

따로 빼는 것 보다는 위에서 삼항 연산자로 하는건 어떨까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 수정했습니다 ~!

Copy link
Contributor

@ymkim97 ymkim97 left a comment

Choose a reason for hiding this comment

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

👍👍👍👍

@fakerdeft fakerdeft merged commit b5cf331 into develop Feb 14, 2025
2 checks passed
@fakerdeft fakerdeft deleted the fix/TNT-253 branch February 14, 2025 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bugfix 버그 수정 ♻ refactor 기존 코드 리팩토링 ✅ test 테스트 코드 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

회원 - 트레이니 회원 탈퇴 시 식단 데이터 삭제 추가
2 participants