Skip to content

fix(card-deposit): drop Onchain badge from card-deposit error toasts#2040

Merged
MusabShakeel576 merged 1 commit into
masterfrom
claude/fix-borrow-savings-error-nbb0K
May 4, 2026
Merged

fix(card-deposit): drop Onchain badge from card-deposit error toasts#2040
MusabShakeel576 merged 1 commit into
masterfrom
claude/fix-borrow-savings-error-nbb0K

Conversation

@MusabShakeel576
Copy link
Copy Markdown
Contributor

Summary

The default toast badge text is 'Onchain', which is misleading on the "Deposits not available" / "External deposits not available" errors that fire from card-deposit forms (the badge appears inside the toast next to the close button — see screenshot in the linked thread). Pass props: { badgeText: '' } so no badge is rendered for these toasts.

Sites updated:

  • components/Card/CardDepositInternalForm.tsx (Borrow + Savings/Wallet submit handlers)
  • components/Card/CardDepositExternalForm.tsx (External wallet submit)
  • hooks/useCardDeposit.ts (testnet same-chain deposit)

Test plan

  • Trigger any of the card-deposit "not available" errors → toast renders without the Onchain badge.
  • Other onchain toasts (transactions etc.) still render the Onchain badge as before.

https://claude.ai/code/session_018yioW4wRXd4AJjUrU32QLe


Generated by Claude Code

The default toast badge text is 'Onchain', which is misleading on the
'Deposits not available' / 'External deposits not available' errors that
fire from card-deposit forms. Pass an empty badgeText so no badge is
rendered for these toasts.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 4, 2026

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

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-app Ignored Ignored May 4, 2026 7:47am
solid-app-staging Ignored Ignored May 4, 2026 7:47am

Request Review

@MusabShakeel576 MusabShakeel576 merged commit 2aaa304 into master May 4, 2026
4 of 5 checks passed
@MusabShakeel576 MusabShakeel576 deleted the claude/fix-borrow-savings-error-nbb0K branch May 4, 2026 08:02
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