Skip to content

Conversation

@dan437
Copy link
Contributor

@dan437 dan437 commented Mar 28, 2025

Description

  • This PR is mainly refactoring based on the feedback in the main PR that was merged + it updates the STX controller to the latest version.

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Have STX enabled
  2. Be on Ethereum mainnet
  3. Go to the Send feature
  4. You will be able to choose your gas token if you have enough balance for it and its a supported token

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

dan437 added 6 commits March 28, 2025 16:06
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>

Fix a test

Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
@dan437 dan437 requested a review from a team as a code owner March 28, 2025 15:20
@github-actions
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-transactions Transactions team label Mar 28, 2025
@socket-security
Copy link

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@metamask/smart-transactions-controller@16.2.016.3.0 None 0 217 kB metamaskbot

View full report↗︎

@metamaskbot
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

✅ @MetaMask/confirmations

  • app/scripts/controller-init/confirmations/transaction-controller-init.ts

@metamaskbot
Copy link
Collaborator

Builds ready [a51fa31]
UI Startup Metrics (1194 ± 56 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1194109013425612341295
load10399401155541155987
domContentLoaded1033937114553940988
domInteractive16123041627
firstPaint7701321152399224966
backgroundConnect106676910
firstReactRender18134341926
getState10534768
initialActions001001
loadScripts82172393751866898
setupStore7519278
WebpackHomeuiStartup971785128091967993
load82363595869867922
domContentLoaded81762794869862913
domInteractive16125181438
firstPaint47859926344848909
backgroundConnect17114781541
firstReactRender15123041428
getState7420278
initialActions001000
loadScripts81561793868861904
setupStore8516289
FirefoxBrowserifyHomeuiStartup14261215196115014731784
load12801079180414513411630
domContentLoaded12801078180314513401630
domInteractive10341201279097
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2617129132640
firstReactRender24204842433
getState7426379
initialActions001001
loadScripts12561056177314213121603
setupStore7436467
WebpackHomeuiStartup10108431618157910977
load8817441367134841936
domContentLoaded8817431366134841936
domInteractive122331923115142
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20145272331
firstReactRender19163631924
getState1141151679
initialActions001001
loadScripts8647301343130852992
setupStore115611378
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 372 Bytes (0.01%)
  • ui: 0 Bytes (0%)
  • common: 0 Bytes (0%)

@dan437 dan437 requested review from Matt561, klejeune, matthewwalsh0 and pedronfigueiredo and removed request for Matt561 March 28, 2025 16:10
@dan437 dan437 added this pull request to the merge queue Mar 28, 2025
@dan437 dan437 removed the request for review from matthewwalsh0 March 28, 2025 16:17
Merged via the queue into main with commit 4644e1e Mar 28, 2025
151 of 152 checks passed
@dan437 dan437 deleted the submit-batch-update branch March 28, 2025 17:14
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2025
@metamaskbot metamaskbot added the release-12.17.0 Issue or pull request that will be included in release 12.17.0 label Mar 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-12.17.0 Issue or pull request that will be included in release 12.17.0 team-transactions Transactions team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants