Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Expose calendar-uri as property of deleted caldav events#27310

Merged
ChristophWurst merged 1 commit intomasterfrom
enhancement/caldav-trashbin-objects-calendar-id
May 31, 2021
Merged

Expose calendar-uri as property of deleted caldav events#27310
ChristophWurst merged 1 commit intomasterfrom
enhancement/caldav-trashbin-objects-calendar-id

Conversation

@ChristophWurst
Copy link
Member

The trashbin is one report across all calendars of a user. In order to
refresh the data of a single calendar when one of its events is
restored, we need to know what calendar the event belongs to. There
doesn't seem to be a standard property for a URI, so this adds a custom
Nextcloud prop for the calendar-uri.

Required for nextcloud/calendar#3118.

The trashbin is one report across all calendars of a user. In order to
refresh the data of a single calendar when one of its events is
restored, we need to know what calendar the event belongs to. There
doesn't seem to be a standard property for a URI, so this adds a custom
Nextcloud prop for the calendar-uri.

Signed-off-by: Christoph Wurst <[email protected]>
Copy link
Member

@tcitworld tcitworld left a comment

Choose a reason for hiding this comment

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

💪

@ChristophWurst ChristophWurst merged commit 4fc002a into master May 31, 2021
@ChristophWurst ChristophWurst deleted the enhancement/caldav-trashbin-objects-calendar-id branch May 31, 2021 18:01
$result[] = [
'id' => $row['id'],
'uri' => $row['uri'],
'uri' => $row['co.uri'],
Copy link
Member Author

Choose a reason for hiding this comment

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

this was a mistake #27311 :)

return [
'id' => $row['id'],
'uri' => $row['uri'],
'uri' => $row['c.uri'],
Copy link
Member Author

Choose a reason for hiding this comment

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

same #27311

@blizzz blizzz mentioned this pull request Jun 2, 2021
57 tasks
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 feature: dav

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants