Skip to content

Fix open-spaces#1796

Merged
nikoshell merged 4 commits into
ep2026from
ep2026-ics
Jul 13, 2026
Merged

Fix open-spaces#1796
nikoshell merged 4 commits into
ep2026from
ep2026-ics

Conversation

@nikoshell

Copy link
Copy Markdown
Contributor

No description provided.

@nikoshell nikoshell requested a review from gbdlin July 13, 2026 14:40
Comment thread src/utils/openSpaces.ts Fixed
Comment thread src/utils/openSpaces.ts Fixed
Comment thread src/utils/openSpaces.ts Fixed
Comment thread src/utils/openSpaces.ts Fixed
@europython-society

europython-society commented Jul 13, 2026

Copy link
Copy Markdown
Member

Preview available

Key Value
url https://ep2026-ics.ep-preview.click
last update 2026-07-13T15:06:19.484Z

Comment thread src/utils/openSpaces.ts Fixed
Comment thread src/utils/openSpaces.ts Fixed
Comment thread src/utils/openSpaces.ts Fixed
Comment thread src/utils/openSpaces.ts Fixed
Replace two chained regexes (tag stripping + < removal) with
a single /<[^>]*(?:>|$)/g that handles both complete and
incomplete HTML tags as one multi-character pattern.

- <tag> matches via [^>]*>
- <tag without > at end of string matches via [^>]*$
- This satisfies CodeQL's multi-character sanitization rule
The ICS data uses UTC (Z suffix) for DTSTART/DTEND, but the
calendar grid and detail page were displaying the raw UTC hour
(e.g. 08:00) instead of converting to Europe/Warsaw (UTC+2
in July = 10:00 CEST).

Fix:
- Grid: use fmtTime(startISO) for grouping and display labels
- Grid header: label column as 'Time (CEST)'
- Detail page: convert startISO/endISO via toLocaleTimeString
  with Europe/Warsaw timezone
- Clean up: remove unused props (calendarId, timeMin, timeMax)
@nikoshell nikoshell merged commit 3b2cdd7 into ep2026 Jul 13, 2026
9 checks passed
@nikoshell nikoshell deleted the ep2026-ics branch July 13, 2026 15:18
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.

4 participants