-
Notifications
You must be signed in to change notification settings - Fork 1
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
Merge nutrition resolving into nutrition feature branch #133
Conversation
Already done by @uglrl. To do at least something, I ran the migration tests and checked them randomly. I didn't notice anything negative. Well done. |
@Whatsuup are the database insertion methods also called (at the correct place)? I am not aware that that is already done. |
Added env and nutrition update feature |
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.
Good!
you need to run sqlx prepare
so that the ci can run successfully
Backend
Make sure, you have done de following before merging this pull request:
cargo format
andcargo clippy
to format your code and check for improvements.