Skip to content

Conversation

@ryanslatten
Copy link
Contributor

This PR updates the Team Details page to use Angular 14 as well as updating the styling for the Teams section on the Admin page.

Team Details Page
Screenshot 2025-08-12 at 4 52 34 PM

The Team Dashboard was updated to match the new styling
Screenshot 2025-08-12 at 4 53 08 PM

Editing state for the Team
Screenshot 2025-08-12 at 4 53 47 PM

The page is also updated to use a search directory modal instead of having a table of non members and non team events
Screenshot 2025-08-12 at 4 55 00 PM

Testing Steps:

Before testing you should have multiple events and users set up as well as one or more teams.

  • Go to the Team Details page and verify that the page loads fields populated as expected
  • Edit the team by clicking "Edit" and verify that you can successfully change the name or description
  • Add a user by clicking the "Add Member" button and by submitting the modal that opens. Repeat for adding an event
  • Remove a user from the list and repeat for events
  • Verify that you can delete the team and the associated users by clicking "Delete Team" and submitting the modal

Copy link
Contributor

@danbenner-vega danbenner-vega left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a bunch of those bangs in that one file that I'd like looked at. And also the destroy$ thing looks like a mistake?

danbenner-vega
danbenner-vega previously approved these changes Aug 13, 2025
jwmcdougall

This comment was marked as outdated.

@ryanslatten ryanslatten dismissed stale reviews from jwmcdougall and danbenner-vega via ac427b0 August 13, 2025 21:03
RyCarpenter
RyCarpenter previously approved these changes Aug 14, 2025
danbenner-vega
danbenner-vega previously approved these changes Aug 14, 2025
jwmcdougall
jwmcdougall previously approved these changes Aug 14, 2025
Copy link
Contributor

@jwmcdougall jwmcdougall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

cwerthomni
cwerthomni previously approved these changes Aug 15, 2025
Copy link
Contributor

@jwmcdougall jwmcdougall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ryanslatten ryanslatten merged commit 22f3d97 into develop Aug 18, 2025
8 checks passed
@ryanslatten ryanslatten deleted the team-details-upgrade branch August 18, 2025 19:42
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.

6 participants