Skip to content

feat: transaction pay controller #6820

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 36 commits into
base: main
Choose a base branch
from
Draft

feat: transaction pay controller #6820

wants to merge 36 commits into from

Conversation

matthewwalsh0
Copy link
Member

Explanation

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, highlighting breaking changes as necessary
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@socket-security
Copy link

socket-security bot commented Oct 12, 2025 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Added@​metamask/​bridge-status-controller@​49.0.1991007898100
Added@​metamask/​bridge-controller@​49.0.1991007898100
Added@​metamask/​assets-controllers@​79.0.1981009198100

View full report

@matthewwalsh0
Initial commit
b3499c7
@matthewwalsh0
Add publish hook
4b0daf0
@matthewwalsh0
Parse token transfer as required token
673c93e
@matthewwalsh0
Update tokens on transaction change
ec460d0 
Calculate fiat properties.
Update payment token.
@matthewwalsh0
Fetch quotes
cdf9e52 
Calculate source amounts.
@matthewwalsh0
Add update payment token unit tests
b1d1f4b
@matthewwalsh0
Add source amount unit tests
afe1b4a
@matthewwalsh0
Add dust and fee calculation
94da96a
@matthewwalsh0
Add required assets to transaction controller
7f22ff5
@matthewwalsh0
Pass required assets from middleware
5c4734d
@matthewwalsh0
Parse required tokens from required assets
c69f50f 
Add symbol to tokens.
@matthewwalsh0
Add quote metadata
338968b 
Export types.
Update flags.
@matthewwalsh0
Update bridge status controller
60be69f
@matthewwalsh0
Add totals to state
6122dc7
@matthewwalsh0
Support abstract pay strategy
47bf79c
@matthewwalsh0
Add strategy constructor callback
e2b5c97 
Fix unit tests.
@matthewwalsh0
Add bridge strategy
cd6d6f5
@matthewwalsh0
Add bridge unit tests
aec162c
@matthewwalsh0
Add bridge quotes unit tests
f9e7f2a
@matthewwalsh0
Incude amount in totals
9590ac5
@matthewwalsh0
Handle no source amounts
1094977
@matthewwalsh0
Fix linting
72d2367
@matthewwalsh0
Fix linting
c15bbb2
@matthewwalsh0
Remove unnecessary changes
03f0532
@matthewwalsh0
Add type
e885e88
@matthewwalsh0
Update changelog
c6a4cc0
@matthewwalsh0
Add unit test coverage
5489d31
@matthewwalsh0
Add Relay strategy
1b9a3a3
@matthewwalsh0
Add relay quotes unit tests
4c6b861
@matthewwalsh0 matthewwalsh0 force-pushed the feat/transaction-pay-controller branch from 0e50e56 to 4c6b861 Compare October 20, 2025 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@matthewwalsh0