Merged
Conversation
Collaborator
dnishiyama
commented
Mar 25, 2026
dnishiyama
commented
- Updated user router to allow role changes while preventing profile modifications for users outside the admin's managed home region.
- Refactored user update logic to streamline authorization checks and ensure existing user data is preserved when unauthorized edits are attempted.
- Enhanced tests to validate the new behavior, ensuring that profile fields remain unchanged while roles can still be updated.
…al URL validation schema in the validators package. (#23)
* Update modals and add mobile filter sheet - Updated various modal components to improve layout and responsiveness, including adjustments to class names for better styling. - Introduced a new MobileFilterSheet component for mobile devices to manage filters more effectively. - Integrated MobileFilterSheet into AOsTable, AreasTable, and other relevant components to enhance user experience on smaller screens. - Adjusted filter handling logic to accommodate the new mobile filter interface. * Enhance admin layout with titles and z-index adjustments - Added a title prop to the Layout component for better page identification across various admin pages. - Updated the z-index for the header to ensure proper stacking context. - Adjusted padding for content areas to improve layout consistency. - Modified several admin pages to utilize the new title feature, enhancing user experience and accessibility.
- Introduced EVENT_CATEGORY_OPTIONS and EVENT_CATEGORY_LABEL_MAP constants for better management of event categories. - Updated LocationEventForm, AdminEventTypesModal, and AdminWorkoutsModal to utilize the new constants for displaying event category labels. - Improved label formatting to include category descriptions where applicable.
* Enhance event handling and introduce map event router - Added support for invalidating map queries in AdminDeleteModal and AdminWorkoutsModal. - Updated AdminWorkoutsModal to include loading state and improved event creation/updating feedback. - Refactored event querying to use new map-specific endpoints, enhancing filtering capabilities. - Introduced a new map event router to handle event-related operations with improved structure and filtering options. * Updated the event router to include pagination and sorting capabilities for the `all` endpoint.
… center handling logic (#2)
…modifications for users outside the admin's managed home region. - Refactored user update logic to streamline authorization checks and ensure existing user data is preserved when unauthorized edits are attempted. - Enhanced tests to validate the new behavior, ensuring that profile fields remain unchanged while roles can still be updated.
dnishiyama
requested review from
evanpetzoldt,
pstaylor-patrick and
taterhead247
as code owners
Collaborator
Author
|
I haven't gotten a chance to review this. I'm just passing along to you @taterhead247 . I hope to look sometime this week. But if you're good with it we can merge to staging |
taterhead247
approved these changes
Mar 26, 2026
github-project-automation
bot
moved this from Draft
to Merged
in F3 Nation Tech Pull Requests
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
3 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.