Skip to content
This repository has been archived by the owner on Nov 9, 2024. It is now read-only.

Recipes not loading #301

Closed
HalbautomatikToastgeraet opened this issue Aug 4, 2024 · 2 comments · Fixed by #302
Closed

Recipes not loading #301

HalbautomatikToastgeraet opened this issue Aug 4, 2024 · 2 comments · Fixed by #302

Comments

@HalbautomatikToastgeraet

I can log in but I can't see my recipes. Creating new recipes via the app works without any problems (I can see them in the web interface, but not in the app) and editing and viewing the shopping list also works. Only loading and viewing recipes in general doesn't work.
This happens on both my smartphone and my tablet

Screenshot_mealient

@HalbautomatikToastgeraet
Copy link
Author

HalbautomatikToastgeraet commented Aug 4, 2024

I have this error in the log

2024-08-04T11:24:22.901711Z E j: Caused by: java.time.format.DateTimeParseException: Text '2024-08-04T10:16:54.585617+00:00' could not be parsed, unparsed text found at index 26
2024-08-04T11:24:22.901711Z E j: at java.time.format.DateTimeFormatter.parseResolved0(DateTimeFormatter.java:2053)
2024-08-04T11:24:22.901711Z E j: at java.time.format.DateTimeFormatter.parse(DateTimeFormatter.java:1952)
2024-08-04T11:24:22.901711Z E j: at java.time.LocalDateTime.parse(LocalDateTime.java:487)
2024-08-04T11:24:22.901711Z E j: at java.time.LocalDateTime.parse(LocalDateTime.java:472)
2024-08-04T11:24:22.901711Z E j: at O5.d.c(Unknown Source:26)
2024-08-04T11:24:22.901711Z E j: ... 27 more

@kirmanak
Copy link
Owner

kirmanak commented Aug 4, 2024

Thanks for the report, great finding in the logs! It looks like the backwards compatibility of Mealie API was broken in the recent release v1.11.0. In particular, this PR mealie-recipes/mealie#3847 adds timezone to timestamps that previously didn't have it, and this change breaks the parser in the app.

I will try to come up with something that supports both versions of the response.

@kirmanak kirmanak linked a pull request Aug 4, 2024 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants