-
Notifications
You must be signed in to change notification settings - Fork 0
feat(#527): add meal planner #548
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Add db table: logged_meals Add trpc routes: logMeal, getAllLoggedMeals, deleteLoggedMeal Add page to sidebar: /nutrition Add btn to food-card.tsx for logging a meal
|
Had an issue when clicking a meal tracker add button @MihaiZecheru:
Could this be because of the way you're inserting the meal? I'll look deeper into the issue later, but a good place to start looking in the meantime would be in your insert procedure and/or the db schema for this table. |
|
Ah, I forgot to specify that there has to be a dummy user in the DB for this to work. Create a user in the Users table with ID=TEST_USER |
EightBitByte
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look great and work great on the local device. A few nitpicks when it comes to styling and adding some comments for things TODO later. Overall, great work!
Please fix before Friday. Message me on Discord if you have any questions.
| import { apiTest } from "@api/apiTest"; | ||
| import { describe } from "vitest"; | ||
|
|
||
| // TODO: finish adding tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
For replacing alert with a toast/sonner Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
For replacing alert with toast/sonner Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
Co-authored-by: Jacob Moy <79670488+EightBitByte@users.noreply.github.com>
EightBitByte
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! There are a few things yet to implement but those come later when we're ready.

Summary
Add the meal planner feature (frontend and backend).
Changes
Add db table: logged_meals
Add trpc routes: logMeal, getAllLoggedMeals, deleteLoggedMeal
Add page to sidebar: /nutrition
Add btn to food-card.tsx for logging a meal
Closes #527