Skip to content

fix: update wallet rejection errors in swap widget#1074

Merged
ted-palmer merged 2 commits into
mainfrom
ted/fix-ton-reject-tx-error
Jul 13, 2026
Merged

fix: update wallet rejection errors in swap widget#1074
ted-palmer merged 2 commits into
mainfrom
ted/fix-ton-reject-tx-error

Conversation

@ted-palmer

Copy link
Copy Markdown
Collaborator

No description provided.

@ted-palmer
ted-palmer requested a review from pedromcunha July 10, 2026 20:27
@vercel

vercel Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
relay-sdk-demo Ready Ready Preview, Comment Jul 10, 2026 8:30pm

Request Review

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Claude finished @ted-palmer's task in 27s —— View job


Code Review

  • Gather context (diff, commits, surrounding code)
  • Review changes for correctness, security, error handling, and repo guidelines

Code Review — No issues found.

Reviewed 2 files, 8 lines changed. Checked for correctness, security, error handling, and repo guidelines compliance.

The change broadens wallet-rejection detection in SwapWidgetRenderer.tsx:798-801 by (1) matching the shorter substring 'reject' instead of 'rejected' and (2) lowercasing the error text before comparison. This correctly captures more wallet variants ("Reject", "User Rejected the request", "rejecting", etc.) that previously slipped through the case-sensitive 'rejected' check and fell into the generic error path. Logic and short-circuit ordering are preserved, and the changeset (patch to @relayprotocol/relay-kit-ui) matches the scope of the fix.

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Claude finished @ted-palmer's task in 27s —— View job


📝 Docs Drift Check — No docs impact

This PR does not touch surfaces that require a relayprotocol/relay-docs update.

@ted-palmer
ted-palmer merged commit b4a3f7c into main Jul 13, 2026
6 checks passed
@ted-palmer
ted-palmer deleted the ted/fix-ton-reject-tx-error branch July 13, 2026 18:31
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