Skip to content

재료 즐겨찾기#22

Merged
yunjaeeun merged 2 commits intodevelopfrom
3-2
Dec 14, 2024
Merged

재료 즐겨찾기#22
yunjaeeun merged 2 commits intodevelopfrom
3-2

Conversation

@yunjaeeun
Copy link
Copy Markdown
Contributor

@yunjaeeun yunjaeeun commented Dec 13, 2024

📋 요약

  • 재료 즐겨찾기 등록, 수정, 삭제 구현

🛠 변경 사항

  • 이번 Pull Request에서 작업한 주요 변경 사항은 다음과 같습니다:
    • 새로운 기능 추가 또는 버그 수정
    • 코드 리팩토링 또는 최적화
    • 문서 업데이트

🔗 관련 이슈

📸 스크린샷 또는 GIF (해당되는 경우)

  • UI 변경이 있는 경우 변경 사항을 보여주는 스크린샷이나 GIF를 추가해주세요.

✅ 체크리스트

  • 코드가 정상적으로 동작하는지 테스트했습니다.
  • 관련 문서를 작성하거나 업데이트했습니다. (해당되는 경우)
  • 변경 사항을 반영한 테스트 코드를 추가했습니다.
  • 모든 테스트가 성공적으로 통과했습니다.

📚 추가 참고 사항

1. UserPk를 받아 사용자가 즐겨찾기 한 재료 조회

Related to: #3
1. ingredientBookmarkPk를 받아 사용자가 즐겨찾기 한 재료 삭제

Resolves: #3
@yunjaeeun yunjaeeun self-assigned this Dec 13, 2024
Copy link
Copy Markdown
Contributor

@uni-zun uni-zun left a comment

Choose a reason for hiding this comment

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

기능구현 굿 ,
import * 로 깔끔하게 정리 된 것 굿굿

Copy link
Copy Markdown
Contributor

@LeeGukgeon LeeGukgeon left a comment

Choose a reason for hiding this comment

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

즐겨찾기 조회 삭제 메서드 구현이 잘 되어있는 것 같습니다

Copy link
Copy Markdown
Contributor

@kingkang85 kingkang85 left a comment

Choose a reason for hiding this comment

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

확인했습니다! import 깔끔하게 정리한 부분과 적절한 응답 메세지를 보내주는 것 좋네요!!

Copy link
Copy Markdown
Contributor

@KIM9909 KIM9909 left a comment

Choose a reason for hiding this comment

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

즐겨찾기 기능이 잘 구현된 것 같습니다. 굿굿

@yunjaeeun yunjaeeun merged commit d5f74c5 into develop Dec 14, 2024
kingkang85 added a commit that referenced this pull request Dec 17, 2024
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.

5 participants