Skip to content

fix(card-deposit): surface zod v4 validation errors in deposit form#2037

Merged
MusabShakeel576 merged 1 commit into
masterfrom
claude/cherrypick-deposit-card-minimum-validation-to-master
May 3, 2026
Merged

fix(card-deposit): surface zod v4 validation errors in deposit form#2037
MusabShakeel576 merged 1 commit into
masterfrom
claude/cherrypick-deposit-card-minimum-validation-to-master

Conversation

@MusabShakeel576
Copy link
Copy Markdown
Contributor

Zod 4 exposes parse failures via error.issues, but the deposit form was reading error.errors (the v3 shape), so validationError always resolved to null and the balance / borrow / invalid-amount messages never reached ErrorDisplay. Switch to err.issues[0].message, matching the extraction used in CardWithdrawForm, so users see the actual reason the button is disabled.

https://claude.ai/code/session_015A1KTGTkLJht3CZWifFKZN

Zod 4 exposes parse failures via error.issues, but the deposit form was
reading error.errors (the v3 shape), so validationError always resolved
to null and the balance / borrow / invalid-amount messages never
reached ErrorDisplay. Switch to err.issues[0].message, matching the
extraction used in CardWithdrawForm, so users see the actual reason the
button is disabled.

https://claude.ai/code/session_015A1KTGTkLJht3CZWifFKZN
@vercel
Copy link
Copy Markdown

vercel Bot commented May 3, 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 3, 2026 1:55pm
solid-app-staging Ignored Ignored May 3, 2026 1:55pm

Request Review

@MusabShakeel576 MusabShakeel576 merged commit fa0b1d1 into master May 3, 2026
4 of 5 checks passed
@MusabShakeel576 MusabShakeel576 deleted the claude/cherrypick-deposit-card-minimum-validation-to-master branch May 3, 2026 13:55
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