Skip to content

#14 재료 즐겨찾기 수정#30

Merged
LeeGukgeon merged 1 commit intodevelopfrom
14
Dec 14, 2024
Merged

#14 재료 즐겨찾기 수정#30
LeeGukgeon merged 1 commit intodevelopfrom
14

Conversation

@yunjaeeun
Copy link
Copy Markdown
Contributor

@yunjaeeun yunjaeeun commented Dec 14, 2024

📋 요약

  • 사용자 냉장고 테이블 추가에 따른 기존 로직 수정
    • 재료사용 -> 사용자 냉장고 속 재료 사용

🛠 변경 사항

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

🔗 관련 이슈

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

✅ 체크리스트

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

🛡 테스트 방법

📚 추가 참고 사항

  • 이 PR과 관련해 리뷰어가 알아야 할 추가 내용이 있다면 작성해주세요.

1. 사용자 냉장고 생성에 따른 로직 수정

Resolves: #14
Copy link
Copy Markdown
Contributor

@MinFe4990 MinFe4990 left a comment

Choose a reason for hiding this comment

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

!check
column명 수정에 따른 나머지 다 수정한 것 확인했습니다.

@uni-zun
Copy link
Copy Markdown
Contributor

uni-zun commented Dec 14, 2024

리팩토링 확인했습니다 !

@yunjaeeun yunjaeeun requested a review from MinFe4990 December 14, 2024 14:44
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.

Good~! 확인했습니다.

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.

확인 했습니다. 잘 작성 된 것 같습니다 굿굿 !

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.

재료에서 사용자의 냉장고에 있는 재료로 바뀐 구조 확인했습니다

@LeeGukgeon LeeGukgeon merged commit aa45066 into develop Dec 14, 2024
MinFe4990 added a commit that referenced this pull request Dec 22, 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.

6 participants