Skip to content
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

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

st3iny
Copy link
Member

@st3iny st3iny commented Jun 11, 2024

Summary

The CalDAV RFC says:

In the absence of this property [supported-calendar-component-set], the server MUST accept all component types, and the client can assume that all component types are accepted.

So collections that are created without supported-calendar-component-set should also support journals.

TODO

  • Probably tests

Checklist

@st3iny st3iny added 2. developing Work in progress feature: caldav Related to CalDAV internals labels Jun 11, 2024
@st3iny st3iny self-assigned this Jun 11, 2024
Copy link
Contributor

@miaulalala miaulalala left a 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

@st3iny
Copy link
Member Author

st3iny commented Jun 11, 2024

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 😕

@solracsf solracsf added this to the Nextcloud 30 milestone Jun 18, 2024
@lmouelle
Copy link

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

@st3iny st3iny force-pushed the fix/caldav/default-vjournal branch from 23fc9c5 to fdd778a Compare August 13, 2024 12:46
@st3iny
Copy link
Member Author

st3iny commented Aug 13, 2024

Thanks for the tip! I was finally able to do some testing with GNOME Evolution and it works fine.

The backend handles VJOURNALs out of the box and the Calendar app simply ignores them.

@st3iny st3iny added bug 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 13, 2024
@st3iny st3iny marked this pull request as ready for review August 13, 2024 12:47
@st3iny st3iny requested a review from miaulalala August 13, 2024 12:47
This is mandated by the RFCs.

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
@st3iny st3iny force-pushed the fix/caldav/default-vjournal branch from fdd778a to 82615ce Compare August 13, 2024 13:26
@st3iny st3iny added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 13, 2024
@st3iny st3iny enabled auto-merge August 13, 2024 13:26
@st3iny st3iny merged commit bb70699 into master Aug 13, 2024
169 checks passed
@st3iny st3iny deleted the fix/caldav/default-vjournal branch August 13, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug feature: caldav Related to CalDAV internals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: MKCALENDAR without supported-calendar-component-set doesn't support journals by default
5 participants