feat: Multi Custom Events, persist, select, and manage custom events with URL import#10
Open
Podders wants to merge 10 commits intokfitzgerald:masterfrom
Open
feat: Multi Custom Events, persist, select, and manage custom events with URL import#10Podders wants to merge 10 commits intokfitzgerald:masterfrom
Podders wants to merge 10 commits intokfitzgerald:masterfrom
Conversation
…e (fix missing CustomEventReducer); keep Day display PR unaffected
Podders
changed the title
kfitzgerald
reviewed
Dec 7, 2025
src/raidpal/RaidPalView.js
Outdated
| const unique = []; | ||
| const { time_table: slots, slot_duration_mins, inconsistent_slot_durations=false } = event; | ||
|
|
||
| const toLower = (s) => (typeof s === 'string') ? s.toLowerCase() : ''; |
Owner
There was a problem hiding this comment.
This could probably move up a scope so #93 can use it too
Contributor
Author
There was a problem hiding this comment.
Lifted this out into the Utils file and applied it application wide where it can be used,
I've also done the TODO that automatically shows the closest upcoming slot event in RaidPal before any others too so the user should also see their next slot first and all events in the dropdown ordered by their own slot time.
Introduces a utility function to convert strings to lowercase, ensuring consistency across the application. Improves event selection logic to prioritise events with slots closest to the current time for the logged-in user.
Improves RaidPal event selection logic by considering user slots and proximity to current time. It introduces normalisation functions for user IDs and logins to ensure accurate matching.
Contributor
Author
|
@kfitzgerald Is this one likely to make the cut any time soon? we've got some Team Organised events coming up that would be nice to be able to track them as custom events :) |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This PR adds full support for multiple custom events, decoupled from RaidPal-specific state. Users can load events by pasting a JSON URL or a live RaidPal event URL, which is automatically mapped to the RaidPal API endpoint. Loaded events persist across reloads and are presented under a new “Custom” tab with a dropdown selector and a “Custom Events” label for visual consistency.
A redesigned Custom Event modal provides a tabbed interface to load new events or manage saved ones; the manage view lists saved events with options to select or delete, and includes responsive layout tweaks for small screens. The app safely handles unoccupied slots and missing display names to avoid toLowerCase errors, and automatically switches back to the Followed tab when the final custom event is removed. The implementation introduces a dedicated custom events slice (state, actions, persistence) and leaves existing behavior untouched for non-custom flows.