Skip to content

fix: show empty state on vault fetch error - #37

Merged
collinsezedike merged 1 commit into
mainfrom
fix/vault-error-empty-state
May 10, 2026
Merged

fix: show empty state on vault fetch error#37
collinsezedike merged 1 commit into
mainfrom
fix/vault-error-empty-state

Conversation

@collinsezedike

@collinsezedike collinsezedike commented May 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replaces the red error banner with the EmptyState component when the vault API call fails
  • Error and empty-array cases are now handled by a single condition — no raw error messages shown to users

Test plan

  • Point VITE_API_URL at an unreachable host — vault list shows the empty state card, not an error banner
  • pnpm lint && pnpm typecheck pass

@vercel

vercel Bot commented May 10, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
meridian-web Ready Ready Preview, Comment May 10, 2026 9:58pm

@collinsezedike
collinsezedike merged commit 8e9c55d into main May 10, 2026
4 checks passed
@collinsezedike
collinsezedike deleted the fix/vault-error-empty-state branch May 10, 2026 23:16
@collinsezedike collinsezedike added bug Something isn't working frontend Involves React components, Tailwind styling, or Next.js pages ui Purely presentational work — layout, spacing, visual polish and removed bug Something isn't working frontend Involves React components, Tailwind styling, or Next.js pages ui Purely presentational work — layout, spacing, visual polish labels May 22, 2026
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