Skip to content

fix: replace fabricated inline data on admin reports page with real data wiring#783

Merged
orbitlumen-hue merged 1 commit into
DogStark:mainfrom
Rufai-Ahmed:fix/admin-reports-inline-mock-data
Jul 24, 2026
Merged

fix: replace fabricated inline data on admin reports page with real data wiring#783
orbitlumen-hue merged 1 commit into
DogStark:mainfrom
Rufai-Ahmed:fix/admin-reports-inline-mock-data

Conversation

@Rufai-Ahmed

Copy link
Copy Markdown
Contributor

What

Removes the fabricated inline data on the admin reports page that sits outside the six named MOCK_* constants (the fake [1..5].map user-log rows, the hardcoded 24,592 total-users card, the hardcoded financial YTD stat cards) and wires those panels to typed data through the existing API layer, with loading and empty states matching the other admin pages.

The activity panel uses /users/me/activity since that is the only activity endpoint available, so it is scoped to the signed-in admin and labeled accordingly.

Testing

tsc and eslint clean on the changed file; the remaining tsc errors in the repo are pre-existing in unrelated files (83 on this branch vs 94 on main).

Closes #765

@orbitlumen-hue

Copy link
Copy Markdown
Collaborator

resolve conflicts

@Rufai-Ahmed
Rufai-Ahmed force-pushed the fix/admin-reports-inline-mock-data branch from e6d3732 to bb3770c Compare July 23, 2026 19:32
@Rufai-Ahmed

Copy link
Copy Markdown
Contributor Author

Done, rebased on main and resolved the conflicts. Kept the Print Report wiring from #778; the scheduled tab keeps the empty state since the fabricated rows are what this PR removes. One thing to flag: I applied for #765 through the wave but I have not been assigned yet.

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.

[Frontend] Admin Reports Page Has Extensive Hardcoded Fabricated Data Beyond the Named MOCK_* Constants

2 participants