Skip to content

feat(members): add member deletion, align route-details dialog, standardize tag colors - #38

Draft
DBStirling wants to merge 1 commit into
mainfrom
david/members-added-features
Draft

feat(members): add member deletion, align route-details dialog, standardize tag colors#38
DBStirling wants to merge 1 commit into
mainfrom
david/members-added-features

Conversation

@DBStirling

Copy link
Copy Markdown
Contributor

Summary

Adds the ability to delete (deactivate) members from the members table, aligns the member route-details dialog with the routes page, and standardizes tag colors across members.

Ticket

What changed

  • Added delete action to members-table.tsx with confirmation; wired through features/members/api.ts
  • Added DELETE /api/captains/[id] and DELETE /api/volunteers/[id] API routes with soft-delete support
  • Added POST /api/captains/[id]/reactivate and POST /api/volunteers/[id]/reactivate routes
  • Extended lib/services/captains.ts and lib/services/volunteers.ts with delete/reactivate service methods
  • Aligned route-details-dialog.tsx layout and UX with the routes page
  • Standardized tag/badge colors in members-table.tsx and member-side-panel.tsx

Why

  • Members table lacked any way to remove inactive members
  • The route-details dialog in the members view was visually inconsistent with the routes page
  • Tag colors were inconsistent across member views

Test plan

  • Open the Members page; confirm the delete action appears and triggers a confirmation before removing a row
  • Verify a deleted member no longer appears in the table (soft-delete — reactivation path exists via API)
  • Open a member's route-details dialog; confirm layout matches the routes page dialog
  • Check tag/badge colors are consistent between the members table and side panel

Screenshots

Checklist

  • Tests pass
  • Typecheck / lint clean
  • Coverage holds
  • No secrets or unscoped UI messaging added

Made with Cursor

@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
beach-metro Ready Ready Preview Aug 24, 2026 11:20pm

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.

1 participant