Skip to content

fix(receive): render addCardDisclaimer for offers, not just gift cards#1076

Open
orveth wants to merge 1 commit into
masterfrom
fix/offer-disclaimer
Open

fix(receive): render addCardDisclaimer for offers, not just gift cards#1076
orveth wants to merge 1 commit into
masterfrom
fix/offer-disclaimer

Conversation

@orveth
Copy link
Copy Markdown
Contributor

@orveth orveth commented May 15, 2026

Summary

addCardDisclaimer was previously only rendered inside the purpose === 'gift-card' branch in receive-cashu-token.tsx. Offers (purpose: 'offer') never displayed their disclaimer text.

This PR lifts the disclaimer rendering above the purpose branch so it works the same way for both gift cards and offers. Both duplicate blocks in receive-cashu-token.tsx (~lines 289 and 462) are updated.

Test plan

  • receive a gift-card token from a purpose: 'gift-card' mint that has addCardDisclaimer set — verify text still renders below the card
  • receive an offer token from a purpose: 'offer' mint that has addCardDisclaimer set — verify text now renders below the OfferItem
  • bun run fix:all clean

Triggered by

bob's agicashlaunch offer launch (#1075) — wants disclaimer "Valid at any Square merchant" to render for an offer.

🤖 Generated with Claude Code

@vercel
Copy link
Copy Markdown

vercel Bot commented May 15, 2026

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

Project Deployment Actions Updated (UTC)
agicash Ready Ready Preview, Comment May 15, 2026 4:52pm

Request Review

@supabase
Copy link
Copy Markdown

supabase Bot commented May 15, 2026

This pull request has been ignored for the connected project hrebgkfhjpkbxpztqqke because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@gudnuf gudnuf requested a review from jbojcic1 May 15, 2026 17:23
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.

3 participants