Skip to content

#17 유통기한 알림#45

Merged
kingkang85 merged 2 commits intodevelopfrom
17
Dec 16, 2024
Merged

#17 유통기한 알림#45
kingkang85 merged 2 commits intodevelopfrom
17

Conversation

@yunjaeeun
Copy link
Copy Markdown
Contributor

📋 요약

  • 내 냉장고에 존재하는 재료들의 유통기한 알림
    • 유통기한이 7일, 3일 남은 대상에 따라서 알림 생성
    • 로그인 시 작동되도록 수정 필요

🛠 변경 사항

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

🔗 관련 이슈

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

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

✅ 체크리스트

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

🛡 테스트 방법

📚 추가 참고 사항

1. 남은 유통기한 알람 세팅
   - 서비스 기능 구현중
   - 기능구현 후 로그인에서 비즈니스 로직 이어가도록 수정

Related to: #17
1. 남은 유통기한 알람 구현 완료
유통기한이 7일, 3일 남은 상품에 대해서 알람 발송

2. 로그인 시 작동되도록 수정 필요

Related to: #17
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.

깔끔하고 좋네요 !

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.

if (daysUntilExpiration == 7 || daysUntilExpiration == 3) 이 부분이 이해가 안돼요 남은 유통기한이 3일이거나 7일인 경우에 response에 넣어서 반환하게 되는거 아닌가요? 이거보다 7일 이하 or 3일 이하 조건으로 넣어야 되지 않을까요

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.

확인했습니다!

@kingkang85 kingkang85 merged commit bdfce6f into develop Dec 16, 2024
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