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

Nutrition stats missing #176

Closed
muety opened this issue Jan 20, 2025 · 1 comment · Fixed by #177
Closed

Nutrition stats missing #176

muety opened this issue Jan 20, 2025 · 1 comment · Fixed by #177
Assignees
Labels
Backend The issue lies in the data presented in the App or in the server bug Something isn't working

Comments

@muety
Copy link
Contributor

muety commented Jan 20, 2025

Thankfully, nutrition stats were introduced in #124 to the app. However, I noticed that they're missing quite frequently, even though available on the official website. I can't provide a clear way to "reproduce" this, but here's an exemplary screenshot. I'd say this happens almost daily for a subset of meals. Perhaps something's wrong with the scraper?

@worldofjoni worldofjoni self-assigned this Jan 22, 2025
@worldofjoni worldofjoni added bug Something isn't working Backend The issue lies in the data presented in the App or in the server labels Jan 22, 2025
@worldofjoni
Copy link
Member

worldofjoni commented Jan 22, 2025

Thanks for this report. This is also something I occasionally noticed. Investigating this, I discovered an issue when updating the nutritional and environmental data in the database: Existing meals without some of this nut/env data (from before this update) did not get an entry created in the new tables for nut/env data.

@worldofjoni worldofjoni linked a pull request Jan 22, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend The issue lies in the data presented in the App or in the server bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants