-
Notifications
You must be signed in to change notification settings - Fork 236
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
Resolves #1835
Based on #2331 and #4310
Closes #2331
Closes #4310
Requires:
TODO