Skip to content

fix: add empty states for transactions and bills views (closes #175)#232

Open
Emmanard wants to merge 1 commit into
kellymusk:mainfrom
Emmanard:fix/issue-175-empty-states
Open

fix: add empty states for transactions and bills views (closes #175)#232
Emmanard wants to merge 1 commit into
kellymusk:mainfrom
Emmanard:fix/issue-175-empty-states

Conversation

@Emmanard

Copy link
Copy Markdown
Contributor

Summary

Wires the existing EmptyStateIllustration component (which was built but unused) into all missing and partial empty state slots across the app.

Changes

  • category-grid.tsx — replaces raw inline SVG with EmptyStateIllustration variant="search"
  • recent-billers.tsx — replaces invisible return null on search no-results with a proper empty state card
  • portfolio-page-client.tsx — replaces bare text with EmptyStateIllustration variant="empty" + Add Funds CTA
  • recent-transactions.tsx — adds defensive empty state guard for when transactions array is empty
  • transaction-history.tsx — adds empty state with Clear Filters CTA, hides table/pagination when no results

Approach

  • Zero new libraries or icon sets introduced
  • All changes are additive only — no existing logic altered
  • Follows the exact pattern established in scheduled-payments.tsx and saved-accounts.tsx
  • clearFilters() helper resets only pre-existing state setters to their original initial values

Closes #175

@vercel

vercel Bot commented Jun 27, 2026

Copy link
Copy Markdown

@Emmanard is attempting to deploy a commit to the kelly musk's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jun 27, 2026

Copy link
Copy Markdown

@Emmanard Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

54. Empty states across app

1 participant