Add calendar feature #16
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add iCalendar (.ics) generation utility functions - Add "Add to Calendar" button to individual session modals - Add "Add All Events to Calendar" button to schedule view - Support for importing events into Google Calendar, Apple Calendar, Outlook, etc. - Properly handle timezone conversion (Argentina time to UTC) - Style buttons to match page theme with semi-transparent white backgrounds
- Replace data URI with direct ICS download for better compatibility - Add dropdown menu to individual events (Google Calendar or ICS options) - Simplify "Add all events" button to underlined text link - Add auto-dismissing tooltip on first event (5 second timeout) - Update calendar utility functions to use streamlined approach
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
Author
|
Converted this into a draft for a while when I'm making some changes. |
- Use TZID format instead of UTC for ICS files - Add VTIMEZONE block to properly define Argentina timezone - Fix Google Calendar URL to use local time with ctz parameter - Rename toISODateString to formatDateTimeForICS for clarity - Ensures events display correctly regardless of user's timezone
- Extract buildEventData() helper function - Remove duplicate event data generation logic - Single source of truth for description, title, location, uid - Reduces code size and improves maintainability
- Add try-catch blocks to all calendar operations - Validate session data before processing - Protect DOM manipulation with try-finally in openICS - Detect and handle popup blockers for Google Calendar - Show user-friendly error messages with fallback options - Log all errors to console for debugging - Prevent app crashes from malformed data or failed operations
carletex
approved these changes
Oct 16, 2025
Member
carletex
left a comment
carletex
left a comment
There was a problem hiding this comment.
Thanks for this Port, looking great!!
Removed the tooltip since I wasn't sold on it and tweaked some styling stuff (some of it existing on previous code)
Merging
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add some buttons to add the events to the calendar.