Skip to content

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

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

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

Conversation

@MusabShakeel576
Copy link
Copy Markdown
Contributor

Summary

Cherry-pick of #2040 onto qa.

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 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 cc041bd into qa May 4, 2026
4 of 5 checks passed
@MusabShakeel576 MusabShakeel576 deleted the claude/fix-borrow-savings-error-nbb0K-qa 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