-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
fix(caldav): allow VJOURNAL if no supported component set is specified #45786
Conversation
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.
Have you tested it with our frontend? Will it display? Because \OCA\DAV\CalDAV\CalDavBackend::getCalendarsForUser will give you all supported component sets
Our own client, the Calendar app, should be fine. It ignores everthing other than VEVENT and VTODO. However, I'm still searching for an application that support VJOURNALS to actually test it. Unfortunately, jtxboard on Android can't sync with CalDAV anymore on the free version 😕 |
Jumping into this issue from a davx5 discussion I opened here, the free version of jtx board in f-droid can do CalDAV sync, that's what I've been using. Evolution/GNOME email client also can do CalDAV sync, VJOURNAL shows in the "memos" section of the desktop app |
23fc9c5
to
fdd778a
Compare
Thanks for the tip! I was finally able to do some testing with GNOME Evolution and it works fine. The backend handles |
This is mandated by the RFCs. Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
fdd778a
to
82615ce
Compare
Summary
The CalDAV RFC says:
So collections that are created without
supported-calendar-component-set
should also support journals.TODO
Checklist