Skip to content

Conversation

@miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Sep 17, 2024

  • Resolves: #

Summary

Ignore everything but the specific component types that are defined as valid in the RFC. We should only denormalize data that is actually valid, and skip any invalid data structures. Otherwise we run into the danger of returning trash data.

TODO

  • ...

Checklist

@miaulalala miaulalala requested review from a team, Altahrim, skjnldsv, st3iny, tcitworld and yemkareems and removed request for a team, ChristophWurst and tcitworld September 17, 2024 11:02
@miaulalala miaulalala self-assigned this Sep 17, 2024
@miaulalala miaulalala added 3. to review Waiting for reviews bug feature: caldav Related to CalDAV internals labels Sep 17, 2024
@miaulalala
Copy link
Contributor Author

/backport to stable30

@miaulalala
Copy link
Contributor Author

/backport to stable29

@miaulalala
Copy link
Contributor Author

/backport to stable28

@miaulalala miaulalala force-pushed the fix/filter-for-components-explicitly branch from b90e8f0 to a65cf51 Compare September 17, 2024 14:44
@nickvergessen
Copy link
Member

Could you make the PR description a bit more verbose?
Any issue that is fixed with this? Or example data set that now behaves different?

@miaulalala miaulalala requested a review from kesselb November 25, 2024 11:20
@AndyScherzinger
Copy link
Member

/backport to stable31

@AndyScherzinger AndyScherzinger force-pushed the fix/filter-for-components-explicitly branch from a65cf51 to cd306b8 Compare January 26, 2025 12:24
@SebastianKrupinski SebastianKrupinski force-pushed the fix/filter-for-components-explicitly branch from a8b4ff6 to 548131b Compare February 8, 2025 17:38
@SebastianKrupinski SebastianKrupinski self-requested a review February 8, 2025 18:05
@nickvergessen nickvergessen removed their request for review February 10, 2025 08:38
@susnux susnux added this to the Nextcloud 32 milestone Mar 2, 2025
@SebastianKrupinski SebastianKrupinski marked this pull request as draft May 9, 2025 16:08
auto-merge was automatically disabled May 9, 2025 16:08

Pull request was converted to draft

@SebastianKrupinski SebastianKrupinski force-pushed the fix/filter-for-components-explicitly branch 3 times, most recently from 5716382 to 79c9ed4 Compare May 9, 2025 16:47
@SebastianKrupinski SebastianKrupinski marked this pull request as ready for review May 9, 2025 17:19
This was referenced Aug 22, 2025
This was referenced Sep 2, 2025
This was referenced Sep 25, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
@SebastianKrupinski SebastianKrupinski force-pushed the fix/filter-for-components-explicitly branch from 79c9ed4 to 8f4c562 Compare November 12, 2025 13:41
@SebastianKrupinski SebastianKrupinski force-pushed the fix/filter-for-components-explicitly branch from 8f4c562 to 06fbea0 Compare November 12, 2025 13:46
Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
@SebastianKrupinski SebastianKrupinski force-pushed the fix/filter-for-components-explicitly branch from 06fbea0 to 64377b7 Compare November 12, 2025 13:49
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 backport-request bug feature: caldav Related to CalDAV internals

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

8 participants