Skip to content

#09 재료기반 레시피 추천 #23

Merged
MinFe4990 merged 2 commits intodevelopfrom
09
Dec 22, 2024
Merged

#09 재료기반 레시피 추천 #23
MinFe4990 merged 2 commits intodevelopfrom
09

Conversation

@uni-zun
Copy link
Copy Markdown
Contributor

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

📋 요약

  • 내 냉장고의 재료와 매칭률 66퍼센트 이상의 레시피를 추천합니다.

🛠 변경 사항

  • 이번 Pull Request에서 작업한 주요 변경 사항은 다음과 같습니다:
    • 새로운 기능 추가 또는 버그 수정

🔗 관련 이슈

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

image

양파의 경우 유통기한이 지났는데 DB에 있어서 -2일로 뜹니다.
추후엔 유통기한이 지난 재료는 내 냉장고에서 없애는 방식으로 한다면 ? 아니면
지난 재료는 표시하지 않는걸로 처리하겠습니다.

✅ 체크리스트

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

🛡 테스트 방법

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

📚 추가 참고 사항

1. 레시피 추천 페이지를 구성하였습니다.
2. Header 에 레시피로 가는 링크 만들어놨는데 추후 수정 해도됩니다.
3. userPk 기반으로 레시피 추천을 하는데 프론트에서 로그인시 pk 를 어디에 담을지 정해지면 수정 하겠습니다.
Related to : #9
@yunjaeeun
Copy link
Copy Markdown
Contributor

유통기한 띄어쓰기 정도만 수정하면 좋을 것 같네요

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.

check!

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.

로컬 스토리지에 있는 토큰 값 이용하시면 유저 정보 가져올 수 있을 것 같습니다!!

@MinFe4990 MinFe4990 merged commit 8bce525 into develop Dec 22, 2024
@uni-zun uni-zun mentioned this pull request Dec 22, 2024
5 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.

6 participants