Skip to content

fix: MM Pay transaction with paymentOverride and isPostQuote defined when source and target tokens have different number of decimal places#9116

Merged
jpuri merged 2 commits into
mainfrom
perps_withdraw
Jun 15, 2026
Merged

fix: MM Pay transaction with paymentOverride and isPostQuote defined when source and target tokens have different number of decimal places#9116
jpuri merged 2 commits into
mainfrom
perps_withdraw

Conversation

@jpuri

@jpuri jpuri commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Explanation

The PR fixed perps withdraw into money account. These are transactions which have paymentOverride and isPostQuote defined. In those cases token amount was set wrong.

References

Related to https://consensyssoftware.atlassian.net/browse/CONF-1540

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

…when source and target tokens have different number of decimal places
@jpuri jpuri requested a review from a team as a code owner June 15, 2026 03:02
@jpuri jpuri temporarily deployed to default-branch June 15, 2026 03:02 — with GitHub Actions Inactive
@jpuri jpuri enabled auto-merge June 15, 2026 03:03
@jpuri jpuri requested a review from a team as a code owner June 15, 2026 03:05
@matthewwalsh0 matthewwalsh0 requested a review from dan437 June 15, 2026 11:00
@jpuri jpuri added this pull request to the merge queue Jun 15, 2026
Merged via the queue into main with commit f616d2c Jun 15, 2026
375 checks passed
@jpuri jpuri deleted the perps_withdraw branch June 15, 2026 11:59
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.

2 participants