UI migration to PrimeReact: tables, roster matrix, avatars, donation bad debts, exit breakdown#30
Merged
Merged
Conversation
…rimeReact stepper inputs
Tables (PrDataTable wrapper, applies to all data grids):
- Paginator: 10 rows/page default, rows-per-page dropdown up to 100,
with a "{first}–{last} of {totalRecords}" report. Opt-out via
`paginated={false}` (member×month pivot keeps all rows + totals).
- removableSort: third click on a column clears the sort.
- size="small" density retained.
- Drop the redundant Status column from the members directory — it's
already split into Active/Inactive accordion sections; status stays
in the CSV export only. (loans-list keeps its multi-value status.)
Responsiveness:
- Switch four raw tables from `overflow-hidden` (which clipped wide
content on mobile) to `overflow-x-auto` + min-width so they scroll
horizontally: meetings list, reference history editor, member bank
accounts manager, and the members-directory read-only bank table.
Inputs (carried from earlier this session):
- App-wide PrimeReact "stacked" stepper for numeric + currency inputs
(PrAmountInput / new PrNumberInput); per-field whole-rupee steps.
- Accordion header tint + count badge; EMI schedule alignment fixes.
- globals.css: bare-specifier node_modules CSS imports so Lightning CSS
(Tailwind v4) doesn't silently drop the Lara theme.
Verified: tsc --noEmit clean, eslint clean, 244/244 tests, build passes.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…xit breakdown - Member×Month matrix shows every active member (migration 050), with serial column and zero-contribution rows rendered as em-dashes - Member directory avatars cached on members.avatar_url (migration 049), shared Avatar component with initials fallback; quiet inline contacts - Donations transactions table now folds in loan write-offs as bad-debt rows - Member-exit approval shows an itemised settlement breakdown - Member contribution bar chart gains an average reference line Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Summary
Continuation of the PrimeReact UI migration + mobile responsiveness work, plus several dashboard data/UX improvements.
Tables & responsiveness
PrDataTable: paginator, removable sort, mobile card-stacking<input type=number>fields → PrimeReact stacked-stepper currency/number inputs across loan, poll, reference, transaction & bank-account formsMember × Month matrix
migration 050); zero-contribution cells render as em-dashesMember directory
members.avatar_url(migration 049) —profilesRLS blocked reading them there<Avatar>component with initials fallbackDonations
Member exits
Migrations to apply
049_member_avatars.sql050_member_month_matrix_all_active.sqlBoth are idempotent (
create or replace/ additive column). Apply manually in Supabase.Verification
npm run buildnpm test(244 passing)npm run lint(1 pre-existing unrelated warning)🤖 Generated with Claude Code