Skip to content

feat(admin-members): add filterable members table - #119

Open
bds165 wants to merge 2 commits into
feat/admin-authfrom
feat/admin-members-tanstack-table
Open

feat(admin-members): add filterable members table#119
bds165 wants to merge 2 commits into
feat/admin-authfrom
feat/admin-members-tanstack-table

Conversation

@bds165

@bds165 bds165 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor
  • Replaces the admin members HTML table with TanStack Table.
  • Adds filters for years remaining, UoA student status, and returning/new member status.
  • Integrates the existing delete member action into each row with confirmation without manual refresh.

@WilliamTayNZ WilliamTayNZ left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good work — all acceptance criteria met. TanStack wiring, filter controls, and delete integration all look correct. Nice touch hiding isConditionalReturningMember from the table with initialState.columnVisibility while still filtering on it.

One more thing before this merges:

Add an Edit link to the actions column. Chuan's edit member PR (#115) has been merged into feat/admin-auth, you'll have to pull the updated branch and merge it into your own, resolving any merge conflicts. As part of that, add an Edit link alongside the Delete button in the actions column in columns.tsx, linking to /admin/members/[id]/edit.

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.

2 participants