Several changes #29

Closed
nilsbraden wants to merge 5 commits from nilsbraden/nextcloud-cookbook:master into master

Sorry for not splitting this up but I should have created different branches thinking ahead. But as I went along I did these changes:

  1. Fix #17 by making the RecipeRepository thread-safe and internally re-selecting the correct category after refreshing data

  2. Bumped some library versions which of course is optional

  3. Fixed an error I had while parsing JSON-Data with an empty nutrition array

Thanks für sharing this cool app!
Grüße nach München! :)

Sorry for not splitting this up but I should have created different branches thinking ahead. But as I went along I did these changes: 1. Fix #17 by making the RecipeRepository thread-safe and internally re-selecting the correct category after refreshing data 2. Bumped some library versions which of course is optional 3. Fixed an error I had while parsing JSON-Data with an empty nutrition array Thanks für sharing this cool app! Grüße nach München! :)
nilsbraden added 5 commits 1 month ago
Owner

Thanks for your work, that looks good.
But I'm actual rewriting the whole loading code, so I can't merge it with my local dev branch.
I will take ideas from your code in my development, so your work was not for nothing.

Grüße zurück :)

Thanks for your work, that looks good. But I'm actual rewriting the whole loading code, so I can't merge it with my local dev branch. I will take ideas from your code in my development, so your work was not for nothing. Grüße zurück :)
MicMun closed this pull request 4 weeks ago
Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.