Skip to content

Conversation

@riceCakeSsamanKo
Copy link
Contributor

🔥 Related Issue

🏃‍ Task

  • 기존 nullable=false를 연관관계 제거를 위해 nullable = true 변경
  • dto 매개변수 타입을 double에서 Double로 변경. 반환값 중 기존에는 반환될 값이 없는 경우 0이 반환되었지만 Double로 설정하여 null이 반환되도록 수정함.
  • 섭취한 즐겨찾기 음식제거 로직 수정. 예를 들어 dailyBookMarkedRaw의 BookMarkedRaw가 제거되어 null이 된 경우, 해당 dailyBookMarkedRaw는 제거하지 못하게 함. 그 경우에는 log를 띄우도록 함

📄 Reference

  • None

✅ Check List

  • PR의 제목은 팀 내 규칙을 준수하여 알맞게 작성하였는가?
  • Merge 하는 브랜치가 올바른가? (main branch에 실수로 PR 생성 금지)
  • 팀의 코딩 컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 들어가지는 않았는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • Reviewers, Assignees, Lables, Project, Milestone은 적절하게 선택하였는가?
  • 관련한 issue를 닫아야 하는지 점검해보고 적용했는가?

Copy link
Contributor

@dionisos198 dionisos198 left a comment

Choose a reason for hiding this comment

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

머얌 pr 한지 몰랐네 확인했어 ! 테이블 구조 바꿔줭!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactoring] 각종 수정 사항 수정

2 participants