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

refactor: 꿀조합 목록 조회 API 수정 #44

Merged
merged 6 commits into from
Apr 20, 2024
Merged

Conversation

Go-Jaecheol
Copy link
Contributor

Issue

✨ 구현한 기능

  • 꿀조합 목록 조회 API 수정
    • favoriteCount -> favorite(boolean)
    • content 추가
    • 상품 image 추가
    • 상품 averageRating 추가

📢 논의하고 싶은 내용

  • 상품이 포함된 꿀조합 목록을 조회하는 API에서도 같은 DTO를 사용하고 있고, 디자인 시안에서도 같은 방식으로 사용되는 것 같아서 일단 같이 수정했습니다.
    근데 보면 볼수록 얘는 product보다는 recipe에 있어야 될 것 같은데, 지금 도메인 전체적으로 수정하고 있다고 해서 일단 인지만 해두고 도메인 수정할 때 같이 수정하도록 할까요?

🎸 기타

X

⏰ 일정

  • 추정 시간 : 2h
  • 걸린 시간 : 2h

Copy link

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 5ad422f.

Copy link
Member

@70825 70825 left a comment

Choose a reason for hiding this comment

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

코드 다운 받아서 확인해봤는데 고칠 부분 없는 것 같아요 완전 고생했습니다 👍

@Go-Jaecheol Go-Jaecheol merged commit d3665bd into develop Apr 20, 2024
3 checks passed
@Go-Jaecheol Go-Jaecheol deleted the feat/issue-41 branch April 20, 2024 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

꿀조합 목록 조회 API 수정
2 participants