Skip to content

19 랜덤 레시피 추천#40

Merged
uni-zun merged 4 commits intodevelopfrom
19-랜덤-레시피-추천
Dec 16, 2024

Hidden character warning

The head ref may contain hidden characters: "19-\ub79c\ub364-\ub808\uc2dc\ud53c-\ucd94\ucc9c"
Merged

19 랜덤 레시피 추천#40
uni-zun merged 4 commits intodevelopfrom
19-랜덤-레시피-추천

Conversation

@uni-zun
Copy link
Copy Markdown
Contributor

@uni-zun uni-zun commented Dec 16, 2024

📋 요약

  • 자신이 가진 재료와 무관하게 랜덤한 레시피를 받을 수 있는 서비스 ( 등록되어 있는 레시피 한해서)
  • RecipeRecommendResponse 를 활용하였습니다. 이름, 내용, 조리시간 정도만 필요할 것 같아서 !

🛠 변경 사항

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

🔗 관련 이슈

  • 이 PR로 해결되는 이슈: #[이슈 번호]
  • 관련된 이슈: #[이슈 번호]

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

image

✅ 체크리스트

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

🛡 테스트 방법

  • 변경 사항을 확인하기 위한 테스트 방법을 단계별로 작성해주세요:
    1. 브랜치를 로컬로 가져옵니다.
    2. 아래 명령어를 사용해 애플리케이션 또는 테스트를 실행합니다:
      # 예시 명령어
      ./run_tests.sh
      
    3. 예상 동작이나 출력 결과를 확인합니다.

📚 추가 참고 사항

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

@yunjaeeun
Copy link
Copy Markdown
Contributor

재료까진 들어가도 괜찮아보입니다.

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

@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

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.

랜덤 추천 확인했습니다.

1. 레시피에 필요한 전체 재료도 가져옴
2. 필수재료 인지는 boolean 으로 구별
Related to : #19-2
@uni-zun uni-zun merged commit e5c021f into develop Dec 16, 2024
@uni-zun uni-zun mentioned this pull request Dec 22, 2024
7 tasks
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.

5 participants