Skip to content
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

Fix nutrition #47

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix nutrition #47

wants to merge 3 commits into from

Conversation

david-askari
Copy link

Fix nutrition import into Mealie by matching their API spec:

"nutrition": {
    "calories": "string",
    "carbohydrateContent": "string",
    "cholesterolContent": "string",
    "fatContent": "string",
    "fiberContent": "string",
    "proteinContent": "string",
    "saturatedFatContent": "string",
    "sodiumContent": "string",
    "sugarContent": "string",
    "transFatContent": "string",
    "unsaturatedFatContent": "string"
}

Black and Flake8 have opposing opinions on the class definition of
'RecipeFood'. Ignore the relevant Flake8 rule [1] as stated in the Black
docs [2] and follow PEP8.

Also don't break the long line on 232.

[1]: https://www.flake8rules.com/rules/E701.html
[2]: https://black.readthedocs.io/en/stable/guides/using_black_with_other_tools.html#e701-e704
Match nutrition properties to Mealie API [1] and fix nutrition import of
carbs, fat and protein.

[1]: https://docs.mealie.io/api/redoc/#tag/Recipe:-CRUD/operation/update_one_api_recipes__slug__put
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.

1 participant