Skip to content

Conversation

@georgehrke
Copy link
Member

@georgehrke georgehrke commented Sep 17, 2019

The DAV Sharing plugin enforces an (arbitrary) check for the calendar to be writeable: https://github.com/nextcloud/server/blob/master/apps/dav/lib/DAV/Sharing/Plugin.php#L166

This should ideally be fixed, but is something to look into for Nextcloud 18. In the meantime, properly mark the birthday calendar as not shareable.

@georgehrke georgehrke added the 2. developing Work in progress label Sep 17, 2019
@georgehrke
Copy link
Member Author

/backport to stable17

@georgehrke georgehrke added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 17, 2019
@georgehrke georgehrke changed the title properly mark birthday calendars as not writable for now properly mark birthday calendars as not shareable for now Sep 17, 2019
Signed-off-by: Georg Ehrke <developer@georgehrke.com>
@georgehrke georgehrke force-pushed the bugfix/noid/properly_mark_birthday_calendars_as_not_sharable branch from a0d4936 to b0a3782 Compare September 17, 2019 10:21
@rullzer rullzer added this to the Nextcloud 18 milestone Sep 17, 2019
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@rullzer rullzer merged commit 85610d9 into master Sep 17, 2019
@rullzer rullzer deleted the bugfix/noid/properly_mark_birthday_calendars_as_not_sharable branch September 17, 2019 16:03
@backportbot-nextcloud
Copy link

backport to stable17 in #17184

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants