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

Event results might not be valid ICS #139

Open
1 of 3 tasks
niccokunzmann opened this issue Apr 9, 2024 · 0 comments
Open
1 of 3 tasks

Event results might not be valid ICS #139

niccokunzmann opened this issue Apr 9, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@niccokunzmann
Copy link
Owner

niccokunzmann commented Apr 9, 2024

Describe the bug

When I have an event with the attribute DURATION (https://www.rfc-editor.org/rfc/rfc5545#section-3.8.2.5), then the value DTEND is still set. This makes it an invalid event for to_ical(): DTEND and DURATION MUST not occur at the same time. This might not be so much of an issue but it could be. This should be checked.

To Reproduce

Get the events from the example with duration
See that DTEND and DURATION are both possible.

ICS file

https://github.com/niccokunzmann/python-recurring-ical-events/blob/main/test/calendars/duration-edited.ics

Expected behavior

I would opt for removing DURATION like RDATE and such, as written in https://github.com/niccokunzmann/python-recurring-ical-events?tab=readme-ov-file#events-as-list

Console output

Version:

Additional context

Suggested implementation


We're using Polar.sh so you can upvote and help fund this issue. We receive the funding once the issue is completed & confirmed by you. Thank you in advance for helping prioritize & fund our work. Fund with Polar
@niccokunzmann niccokunzmann added the bug Something isn't working label Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant