Bug: Meal plan shows calories per number of base servings and not per serving as stated
See original GitHub issueEDIT by @berrnd: TODO for this issue: Show the kcal on the recipe page and the meal plan always per 1 serving.
I’ve noticed that “kcal per servings” are not showing correct in the meal plan “recipe card”, it always displays calories per number of base servings from recipe. This would not affect demo data in demo environment since all the recipes have 1 as base servings. its easy to produce this error in the demo environment just by doubling the base servings for one recipe as shown below. Screenshots are taken from prelease out of Version pre-release-3d82c9a
Meal plan shows 984 kcal per servings. Should in this case be 492 since we are only planing to make 1 serving
Recipe shows 984 kcal for 2 desired servings which is correct. (I think it would be more useful to show kcal per 1 serving here as well but that’s besides the point in this ticket)
Recipe shows 492 kcal for 1 desired servings which is correct
Issue Analytics
- State:
- Created 3 years ago
- Comments:5 (3 by maintainers)
Top GitHub Comments
Desired servings = the amount of servings you choose on the recipe page to calculate the ingredients/kcal/etc. accordingly. Meal plan servings = the amount of servings of the recipe you have added to the meal plan on a specific day.
Yes, maybe it would be best to display always (recipe page + meal plan) the kcal per 1 serving - let’s change that.