Skip to content

Quote swaps and sells above the wallet balance - #6308

Open
j0ntz wants to merge 2 commits into
developfrom
jon/ignore-balances-for-quoting
Open

j0ntz wants to merge 2 commits into
developfrom
jon/ignore-balances-for-quoting

Conversation

@j0ntz

@j0ntz j0ntz commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Description

Asana task

Users could not shop for swap or sell quotes until they had moved funds into Edge: an amount above the wallet balance hid Next. Quotes now surface at any amount, and the transaction is blocked with a warning.

Swap

  • The Exchange scene keeps Next visible with an Insufficient Funds warning, and requests quotes with allowInsufficientFunds (never for a max amount).
  • The confirmation scene shows a warning card and disables the slider with the text Insufficient Funds whenever a quote is short, whether the plugin flagged it or the amount exceeds the balance. A guard in the slide handler backs this up.
  • An unknown network fee reads N/A in place of a false zero.
  • When any funded quote exists, best-quote selection only picks from funded quotes. A user with a marginal balance does not land on a short quote that happens to pay more.

Sell

  • The Sell scene requests quotes above the balance and shows a warning card. Next also stays tappable after filling the crypto amount, the case in the dependent task.
  • The Select Payment Method scene shows the warning. Pressing a quote shows a toast and stays on the scene, and no provider opens.

Buy is unchanged. Wallet-less shopping is out of scope, since a zero-balance wallet is enough to shop.

A shopping quote creates a real provider order, the same as a funded quote does, including Houdini's one-order-per-minute slot.

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

Test evidence

6e0a911
Quote sells above the wallet balance
2026-11-01

1. exchange amount above balance keeps next

2. swap quote warns fee row na slider locked

3. swap providers quote short amount

4. sell amount above balance keeps next

5. sell quotes surface with warning

6. sell quote press blocked with toast

7. funded swap quote can slide

8. funded swap success

9. zero balance wallet amount keeps next

10. zero balance swap quote warns slider locked

11. zero balance sell amount keeps next

12. zero balance sell quotes surface with warning

13. zero balance sell quote press blocked with toast

Awaiting publish: edge-core-js@2.55.0 (npm latest when applied: 2.54.0)

j0ntz added 2 commits October 31, 2026 19:03
Next stays available for an amount above the balance and asks for quotes
with allowInsufficientFunds. The confirmation scene warns and disables the
slider on a quote the wallet cannot pay for, and prefers a funded quote when
one exists. A missing fee on such a quote reads N/A rather than zero.
A sell amount above the balance no longer stops the quote request, so the
option scene lists quotes. The amount scene warns instead of showing an
error, and pressing a quote the wallet cannot pay for shows a toast rather
than opening the provider. A quote is unfunded when its crypto amount
exceeds the balance, which also covers amounts entered in fiat.
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-11-01T02:39:55.936457Z
We couldn’t complete this review. Comment @codex review to try again.
6e0a911 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@j0ntz j0ntz added the awaiting-dep-publish CI is red until a dependency publishes. Excuses CI only; see the PR body for the awaited package. label Nov 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-dep-publish CI is red until a dependency publishes. Excuses CI only; see the PR body for the awaited package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant