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

iCal file not importable #8498

Closed
1 task done
larseggert opened this issue Feb 4, 2025 · 3 comments · Fixed by #8506
Closed
1 task done

iCal file not importable #8498

larseggert opened this issue Feb 4, 2025 · 3 comments · Fixed by #8506

Comments

@larseggert
Copy link
Collaborator

Describe the issue

On https://datatracker.ietf.org/wg/scone/meetings/, the ical file for the interim, i.e., https://datatracker.ietf.org/meeting/interim-2025-scone-01/sessions/scone.ics, downloads a file that Apple Calendar won't import.

scone.ics.zip

Code of Conduct

@larseggert larseggert added the bug Something isn't working label Feb 4, 2025
@jennifer-richards
Copy link
Member

Ew - we have a few problems there. The one causing the failure is the None on the last line here:

BEGIN:VCALENDAR
VERSION:2.0
METHOD:PUBLISH
PRODID:-//IETF//datatracker.ietf.org ical agenda//EN
NoneBEGIN:VEVENT
[file continues...]

We have some other lint (lines not folded, incorrect line terminators) that are probably tolerated, but should be fixed. Those should probably be added to the refactor-it-all issue if they're not fixed here #5393

@jennifer-richards jennifer-richards removed the contrib-medium Medium dev difficulty for contribution label Feb 6, 2025
@jennifer-richards
Copy link
Member

The glitch here is caused by the meeting time zone being "GMT" rather than "UTC". The None in the ical is caused by {{ schedule.meeting.vtimezone }} being interpolated, and GMT has no vtimezone(). In the short term, we need to disable GMT as a choice.

In the longer term, we need to fix our timestamps for the UTC case - RFC 5545 requires special handling of those.

@jennifer-richards
Copy link
Member

Correction: we already fixed the handling of UTC timestamps. PR #8506 handles GMT as a synonym for UTC as a meeting time zone and should fix this issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants