-
Notifications
You must be signed in to change notification settings - Fork 241
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
feat: add a setting for the default calendar url #5790
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #5790 +/- ##
============================================
- Coverage 24.19% 24.13% -0.07%
Complexity 423 423
============================================
Files 243 243
Lines 11005 11038 +33
Branches 1824 1833 +9
============================================
+ Hits 2663 2664 +1
- Misses 8342 8374 +32
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
8b83768
to
ba8125f
Compare
ba8125f
to
3bfa8c8
Compare
3bfa8c8
to
c130fad
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since it requires nextcloud/server#43745, shouldn't this only be showed on NC 29+ ?
Good catch! I'll add a version check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense
29686c0
to
70695f9
Compare
Co-authored-by: Lukas Boersma <[email protected]> Co-authored-by: szaimen <[email protected]> Signed-off-by: Richard Steinmetz <[email protected]>
70695f9
to
7a307bb
Compare
Thanks to everyone involved for adding this feature. I just started to use NectCloud Calendar and have exact that situation where I don't want to have the personal calendar as default. Unfortunately the setting does not work as expected. Not sure if this is the right place to report this but I will give it a go. We use a shared calendar as a group. The calendar was created by the NextCloud admin user and made available to a group of people (group defined in our NextCloud). People see the shared calendar and add events but don't seem to be able to set it as default calendar - see screenshots below. Admin is able to set this shared calendar as default but still is not used on new events. NextCloud Server version: 30.0.2 |
@Sebastian-Roth Thanks for the report. Yes, this is working as intended. Please refer to #5996 and #5942 for some more information. TL;DR: Some users were confused by the changed default option in the calendar picker as there are some restrictions on which calendars can be picked as the default calendar picker vs in the event calendar picker. Perhaps someone could investigate if can revert the revert now as the support for shared calendars in the backend has been improved since. A note for the future: Please open a new ticket and reference related pull requests and tickets there. Posting in PRs pings a lot of (unrelated) people and sometimes is lost as the pull request is already closed and some people might skip the notification. Thanks :) |
Resolves #1835
Based on #2331 and #4310
Closes #2331
Closes #4310
Requires:
TODO