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

broken fix #1370

Closed
wants to merge 3 commits into from
Closed

broken fix #1370

wants to merge 3 commits into from

Conversation

pbiering
Copy link
Collaborator

@pbiering pbiering commented Mar 2, 2024

#1316

Replace code which is not permitting one-time occurance of VEVENT and VJOURNAL by using a dictionary.

Also had to remove the duplicate UID verification, because VEVENT and VJOURNAL have different UID usually.

@pbiering pbiering changed the title fix for https://github.com/Kozea/Radicale/issues/1316 fix for Worker thread crashes if a VCALENDAR component contains both VEVENT and VJOURNAL components Mar 2, 2024
@pbiering pbiering added the bug label Mar 2, 2024
@pbiering pbiering added this to the 3.1.9 milestone Mar 2, 2024
@pbiering
Copy link
Collaborator Author

pbiering commented Mar 2, 2024

this breaks somehow https://github.com/Kozea/Radicale/blob/master/radicale/tests/static/event2.ics, have to investigate next

@pbiering pbiering closed this Mar 2, 2024
@pbiering
Copy link
Collaborator Author

pbiering commented Mar 2, 2024

broken

@pbiering pbiering removed the bug label Mar 2, 2024
@pbiering pbiering deleted the fix-for-1316 branch March 2, 2024 11:30
@pbiering pbiering changed the title fix for Worker thread crashes if a VCALENDAR component contains both VEVENT and VJOURNAL components broken fix Mar 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant