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

Add calendar object creation API to OCP #29188

Merged
merged 1 commit into from
Oct 15, 2021
Merged

Conversation

miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Oct 12, 2021

This PR adds a public create method to the OCP calendar API.

This change contributes the first part of #20154 but without the hassle of creating a completely new API, but rather extending what we already have. This means apps that provide calendars can now adapt to this and optionally allow other app devs to write to these calendars.

For simplicity the written calendar object is handed over as a string. Therefore we don't have to define a complex type that can contain the many properties a calendar object can have, nor do we leak the internal sabre types.

This change is required for nextcloud/calendar#3478 so that appointment events can be written to a user's calendar.

@miaulalala

This comment has been minimized.

lib/public/Calendar/Exceptions/CalendarException.php Outdated Show resolved Hide resolved
lib/public/Calendar/IExtendedCalendar.php Outdated Show resolved Hide resolved
lib/public/Calendar/IExtendedCalendar.php Outdated Show resolved Hide resolved
lib/public/Calendar/IExtendedCalendar.php Outdated Show resolved Hide resolved
lib/public/Calendar/IExtendedCalendar.php Outdated Show resolved Hide resolved
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Oct 14, 2021
@ChristophWurst ChristophWurst added this to the Nextcloud 23 milestone Oct 14, 2021
apps/dav/lib/CalDAV/CalendarImpl.php Outdated Show resolved Hide resolved
apps/dav/lib/CalDAV/CalendarImpl.php Outdated Show resolved Hide resolved
@kesselb
Copy link
Contributor

kesselb commented Oct 14, 2021

image

@ChristophWurst ChristophWurst changed the title Add create method to extended calendar implementation Add calendar object creation API to OCP Oct 15, 2021
Signed-off-by: Anna Larch <anna@nextcloud.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst removed the 3. to review Waiting for reviews label Oct 15, 2021
@ChristophWurst ChristophWurst added the 4. to release Ready to be released and/or waiting for tests to finish label Oct 15, 2021
@ChristophWurst ChristophWurst mentioned this pull request Oct 15, 2021
16 tasks
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 enhancement feature: dav integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants