-
Notifications
You must be signed in to change notification settings - Fork 95
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
Don't break trashbin if calendar is undefined #1873
Conversation
Signed-off-by: Raimund Schlüßler <raimund.schluessler@mailbox.org>
Codecov Report
@@ Coverage Diff @@
## master #1873 +/- ##
=========================================
Coverage 30.44% 30.44%
Complexity 41 41
=========================================
Files 66 66
Lines 3134 3134
Branches 585 586 +1
=========================================
Hits 954 954
Misses 2180 2180 |
I will merge this now. I created another issue for the second half of the fix: #1875. |
Ah, I believe I am experiencing this issue. My trash bin fails to load right now. I think it might be caused by the issue that this merge fixed. |
The fix will come with the next release. In the meantime you might be able to delete the event that breaks the Tasks trash bin from the Calendar app trash bin. |
…s unavailable Fix is very similar to nextcloud/tasks#1873 Closes #3894 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
…s unavailable Fix is very similar to nextcloud/tasks#1873 Closes #3894 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
…s unavailable Fix is very similar to nextcloud/tasks#1873 Closes #3894 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
This currently only contains the first half of the fix. The trash bin will now properly load and show all deleted items. Deletion also works fine.
But, for deleted events from calendars that only support events (not tasks),
Unknown calendar
will be shown. This is because these calendars are not loaded by the Tasks app by default.In order to make this work properly, we have to load all calendars from the server when opening the trash bin. This also covers calendars that were newly created after the initial load of the app.
Closes #1803.