Skip to content

fix: sPinto unwrap output and send tokens UX issues#420

Merged
fr1jo merged 1 commit intomainfrom
fix/unwrap-spinto-and-send-tokens-ux
Mar 21, 2026
Merged

fix: sPinto unwrap output and send tokens UX issues#420
fr1jo merged 1 commit intomainfrom
fix/unwrap-spinto-and-send-tokens-ux

Conversation

@fr1jo
Copy link
Copy Markdown
Contributor

@fr1jo fr1jo commented Mar 21, 2026

Summary

  • sPinto Unwrap: Fix output showing incorrect PINTO amount (was capped by existing deposit balance instead of actual previewRedeem quote) and stalk/seeds calculated from existing crates instead of new deposit rewards
  • Send Tokens: Fix "ExternalWallet" camelCase on confirmation page, move transfer notice to only appear after both address and destination type are selected, require acknowledgment for both External and Farm Wallet sends, stop resetting checkbox when switching destination type

Test plan

  • Unwrap sPinto with "Unwrap as PINTO deposit" toggled on — verify output shows correct PINTO amount, stalk gained, and seeds gained
  • Unwrap sPinto with toggle off — verify output shows correct PINTO amount matching the previewRedeem quote
  • Send tokens flow — verify warning notice only appears after entering address AND selecting External/Farm Wallet
  • Send tokens flow — verify switching between External/Farm Wallet doesn't reset the acknowledgment checkbox
  • Send tokens flow — verify acknowledgment is required for both External and Farm Wallet
  • Send tokens confirmation page — verify "External Wallet" displays with proper spacing

🤖 Generated with Claude Code

- Fix sPinto unwrap showing incorrect PINTO amount (capped by existing
  deposit balance instead of actual previewRedeem quote)
- Fix unwrap-to-silo stalk/seeds calculated from existing crates instead
  of new deposit rewards
- Fix "ExternalWallet" camelCase on send confirmation page
- Move transfer notice to appear only after both address and destination
  type are selected
- Require acknowledgment checkbox for both External and Farm Wallet sends
- Stop resetting acknowledgment checkbox when switching destination type

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 21, 2026

Deploy Preview for pintomoney ready!

Name Link
🔨 Latest commit 4746c93
🔍 Latest deploy log https://app.netlify.com/projects/pintomoney/deploys/69beca31e73cda000849f8b1
😎 Deploy Preview https://deploy-preview-420.preview.pinto.money
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@fr1jo fr1jo merged commit c7eb804 into main Mar 21, 2026
6 checks passed
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