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

fix(caldav): limit property length #47413

Merged
merged 1 commit into from
Aug 22, 2024
Merged

Conversation

miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Aug 22, 2024

Summary

Make longer properties searchable for the property index

Checklist

@miaulalala miaulalala added the 3. to review Waiting for reviews label Aug 22, 2024
@miaulalala miaulalala added this to the Nextcloud 31 milestone Aug 22, 2024
@miaulalala miaulalala self-assigned this Aug 22, 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

Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

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

Won’t that apply to event description or stuff like that which might need more than 255 chars?

@miaulalala
Copy link
Contributor Author

Won’t that apply to event description or stuff like that which might need more than 255 chars?

it's just the search index - the data itself is stored in oc_calendar_objects

@miaulalala miaulalala merged commit 639caf6 into master Aug 22, 2024
171 checks passed
@miaulalala miaulalala deleted the fix/limit-property-values branch August 22, 2024 11:17
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