Skip to content

Show actual executed swap amounts after successful swap - #23

Merged
mcintyre94 merged 1 commit into
mainfrom
claude/issue-10-swap-results-L1JPG
Feb 10, 2026
Merged

Show actual executed swap amounts after successful swap#23
mcintyre94 merged 1 commit into
mainfrom
claude/issue-10-swap-results-L1JPG

Conversation

@mcintyre94

Copy link
Copy Markdown
Owner

Summary:
∙ After a successful swap, display the actual input and output token amounts (with icons and symbols) instead of just a generic “Swap successful!” message
∙ Passes inputToken and outputToken props to SwapButton so it can parse and display the executed amounts from the swap result
∙ Uses formatTokenAmount for consistent number formatting in the success output

Test plan:
∙ Perform a token swap and verify the success message shows the actual swapped amounts with token icons and symbols
∙ Verify the “View transaction” link still works correctly
∙ Verify the UI displays correctly for tokens with and without logos

@vercel

vercel Bot commented Feb 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
swapsies Ready Ready Preview, Comment Feb 10, 2026 10:58pm

@claude

claude Bot commented Feb 8, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

After a swap executes, display the actual input and output amounts
from Jupiter's execute response instead of just "Swap successful!".
Shows token logos, formatted amounts, and symbols for both sides
of the swap, with the transaction link below.

Closes #10

https://claude.ai/code/session_01M8UUYNBVgvkd9Wkjn7qJmY
@mcintyre94
mcintyre94 force-pushed the claude/issue-10-swap-results-L1JPG branch from 1448c9f to c67a56c Compare February 10, 2026 22:57
@mcintyre94
mcintyre94 merged commit fea66c0 into main Feb 10, 2026
4 checks passed
@mcintyre94
mcintyre94 deleted the claude/issue-10-swap-results-L1JPG branch February 10, 2026 23:06
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