Closed sanchitc05 closed 1 week ago
Visit the deploys page to approve it
Name | Link |
---|---|
Latest commit | f5e76fa23296ded7a348436144ba96135ea3dc26 |
@sanchitc05 can you please add the video of the output here
Also can you test with the case of date change so that every day different recipe is shown?
Sure thing @AlfiyaSiddique I'll share soon
@sanchitc05 any updates?
Yeah @AlfiyaSiddique I'll just do it by today night or tomorrow morning, I had been busy in the past days
Sure @sanchitc05 take your time.
@AlfiyaSiddique Review this and merge if possible
@AlfiyaSiddique Review this and merge if possible
@MastanSayyad the change is not completed yet I think; no update from the contributor's side. Can't merge without testing
@sanchitc05 do the needful, As the PA is active make sure you contact her before you contact us.
@AlfiyaSiddique TBH my pc got a blue screen error and it is out for repair, I took my friend's laptop but could not run it properly as I was getting various dependencies errors. I would just request you to see the code files, I assure you the logic is right and I have integrated it with react so that the output can be seen under the best dishes section. If you want to check the output you could just merge it and then check it directly deployed on netlify, in case you need any modifications you can always revert my pr and I can make changes to it.
Okay @sanchitc05 works
@sanchitc05 There are logical issues in the backend code. I have added a comment please fix it.
@AlfiyaSiddique On it!
@AlfiyaSiddique I'm closing this pr and i will make a new pr doing the necessary modifications that you suggested.
Fixes #328
What does this PR do?
This PR implements a "Recipe of the Day" feature for TastyTrails. It includes:
Backend changes:
featured
andlastFeatured
getRecipeOfTheDay
/recipe-of-the-day
Frontend changes:
RecipeOfTheDay
componentThe feature selects a new recipe each day, avoiding repetition within 7 days when possible. It provides users with daily culinary inspiration directly on the landing page.
Fixes #328 (Implement Recipe of the Day feature)
How should this be tested?
Backend Testing:
/api/recipe-of-the-day
featured: true
Frontend Testing:
Long-term Testing:
Checklist
Required
console.logs