Skip to content

fix(ICalendar): allow to search for event URI#59142

Merged
blizzz merged 1 commit intomasterfrom
fix/caldav_search_for_uri
Mar 23, 2026
Merged

fix(ICalendar): allow to search for event URI#59142
blizzz merged 1 commit intomasterfrom
fix/caldav_search_for_uri

Conversation

@mejo-
Copy link
Copy Markdown
Member

@mejo- mejo- commented Mar 21, 2026

Required for nextcloud/calendar#8048

Checklist

@mejo- mejo- added this to the Nextcloud 34 milestone Mar 21, 2026
@mejo- mejo- self-assigned this Mar 21, 2026
@mejo- mejo- added bug 3. to review Waiting for reviews feature: caldav Related to CalDAV internals labels Mar 21, 2026
@mejo- mejo- requested a review from hamza221 March 21, 2026 17:53
@tcitworld
Copy link
Copy Markdown
Member

Also document the option in ICalendar::search

@mejo- mejo- force-pushed the fix/caldav_search_for_uri branch from 52bbc00 to 6b7e212 Compare March 22, 2026 21:12
@mejo- mejo- changed the title fix(CalDavBackend): allow to search for event URI fix(ICalendar): allow to search for event URI Mar 22, 2026
@mejo-
Copy link
Copy Markdown
Member Author

mejo- commented Mar 22, 2026

Also document the option in ICalendar::search

Thanks, done now ☺️

Copy link
Copy Markdown
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.

An addition to  CalDavBackendTest would be welcomed as well

@mejo- mejo- force-pushed the fix/caldav_search_for_uri branch 2 times, most recently from f81fbd9 to 08b1e01 Compare March 23, 2026 09:00
@mejo-
Copy link
Copy Markdown
Member Author

mejo- commented Mar 23, 2026

An addition to CalDavBackendTest would be welcomed as well

Agreed, also done now ☺️

@mejo- mejo- force-pushed the fix/caldav_search_for_uri branch from 08b1e01 to 7290207 Compare March 23, 2026 09:13
Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the fix/caldav_search_for_uri branch from 7290207 to f34513d Compare March 23, 2026 09:22
@blizzz blizzz merged commit 6f7961f into master Mar 23, 2026
318 of 348 checks passed
@blizzz blizzz deleted the fix/caldav_search_for_uri branch March 23, 2026 13:57
@mejo-
Copy link
Copy Markdown
Member Author

mejo- commented Mar 23, 2026

@tcitworld @hamza221 what do you think about backporting this fix to stable33 at least? Strictly speaking it's an API change, but then it's only an addition and as @tcitworld mentioned in #8048, there's no other implementation using ICalendar::search() yet anyway. And for #8048 to work on stable33 it would be necessary there.

@tcitworld
Copy link
Copy Markdown
Member

I'm obviously not the backport manager, but this seems small enough to be safe.

@mejo-
Copy link
Copy Markdown
Member Author

mejo- commented Mar 23, 2026

/backport to stable33

@nextcloud-bot nextcloud-bot mentioned this pull request Apr 27, 2026
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 bug feature: caldav Related to CalDAV internals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants