Merge nutrition resolving into nutrition feature branch

kronos-et-al / MensaApp

Application for communication and collective exchange of menu information in university catering facilities

MIT License

45 stars 1 forks source link

Merge nutrition resolving into nutrition feature branch #133

Closed Whatsuup closed 8 months ago

Whatsuup commented 8 months ago

Backend

Make sure, you have done de following before merging this pull request:

[x] Your code is documented using doc comments.
[x] You added appropriate logging in your code.
[x] You added unit tests to all your implemented functions.
[x] You ran cargo format and cargo clippy to format your code and check for improvements.

Whatsuup commented 8 months ago

Already done by @uglrl. I opened this PR without checking the current status. I also couldn't estimate how much work it would be. It was only 4 lines of code. Nevertheless, thank you @uglrl!

To do at least something, I ran the migration tests and checked them randomly. I didn't notice anything negative. Well done.

worldofjoni commented 8 months ago

@Whatsuup are the database insertion methods also called (at the correct place)? I am not aware that that is already done.

Whatsuup commented 8 months ago

Added env and nutrition update feature