Skip to content

fix(transaction-pay-controller): submit direct mUSD fiat deposits#9161

Draft
matthewwalsh0 wants to merge 9 commits into
mainfrom
fix/direct-musd-money-account-relay
Draft

fix(transaction-pay-controller): submit direct mUSD fiat deposits#9161
matthewwalsh0 wants to merge 9 commits into
mainfrom
fix/direct-musd-money-account-relay

Conversation

@matthewwalsh0

@matthewwalsh0 matthewwalsh0 commented Jun 16, 2026

Copy link
Copy Markdown
Member

Explanation

Direct mUSD fiat quotes for Money Account deposits can settle directly into the Money Account, so they do not need a Relay quote or Relay execute path.

This PR moves that path to a Ramps-funded, sponsored Money Account vault batch. After fiat settlement, the Fiat strategy refreshes the vault calldata through getAmountData, submits the nested approve/deposit calls through TransactionController:addTransactionBatch, and lets the client publish hook handle 7702/Sentinel sponsorship.

It also keeps a local QA funding path behind client-provided fiat test options so the settlement step can be exercised without creating a real on-ramp order.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

@matthewwalsh0 matthewwalsh0 force-pushed the fix/direct-musd-money-account-relay branch from 321f6ed to 3dde93c Compare June 17, 2026 10:20
@matthewwalsh0 matthewwalsh0 changed the title fix(transaction-pay-controller): fund direct mUSD money account deposits atomically fix(transaction-pay-controller): submit direct mUSD fiat deposits Jun 17, 2026
@matthewwalsh0 matthewwalsh0 force-pushed the fix/direct-musd-money-account-relay branch from 4636045 to b47ac27 Compare June 17, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant