Skip to content

fix: stops generating invalid bridge quotes when no dest addr #34115

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.

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ghgoodreau
Copy link
Contributor

@ghgoodreau ghgoodreau commented Jul 7, 2025

…d required

Description

Open in GitHub Codespaces

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

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.

Copy link
Contributor

github-actions bot commented Jul 7, 2025

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-swaps-and-bridge Swaps and Bridge team label Jul 7, 2025
@metamaskbot
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

🔄 @MetaMask/swaps-engineers (1 files, +11 -3)
  • 📁 ui/
    • 📁 pages/
      • 📁 bridge/
        • 📁 prepare/
          • 📄 prepare-bridge-page.tsx +11 -3

@metamaskbot
Copy link
Collaborator

Builds ready [7b351ec]
UI Startup Metrics (1281 ± 72 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1281113015597213331378
load109796013136211421187
domContentLoaded108995513006211361178
domInteractive19154671841
firstPaint73787123543511241181
backgroundConnect84334916
firstReactRender23165962437
getState1664782233
initialActions40215717
loadScripts833709103159874921
setupStore84294915
WebpackHomeuiStartup22071721282920623202629
load16631303210316117551984
domContentLoaded16561299209116017511963
domInteractive181380141559
firstPaint1536338763168288
backgroundConnect2510272262746
firstReactRender26490423123385402
getState1344271633
initialActions14330539931
loadScripts16511298207915617491922
setupStore197309301828
FirefoxBrowserifyHomeuiStartup14591284195613915281778
load12721109152710013291496
domContentLoaded12721109152710013291495
domInteractive1263449978127333
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2614135172564
firstReactRender27225352734
getState155267321039
initialActions40316319
loadScripts1251109115139913061453
setupStore144194281040
WebpackHomeuiStartup17491454360426418222180
load14901248185614315911741
domContentLoaded14891247185614315911740
domInteractive1103636258106288
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect25185272843
firstReactRender43326154651
getState30616771661232
initialActions8022431615
loadScripts14701223182914115681701
setupStore155236331037
Benchmark value 1281 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1098 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1090 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 24 exceeds gate value 23 for chrome browserify home mean firstReactRender
Benchmark value 17 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 833 exceeds gate value 830 for chrome browserify home mean loadScripts
Benchmark value 1378 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1182 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 17 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 2207 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 265 exceeds gate value 235 for chrome webpack home mean firstReactRender
Benchmark value 14 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 2629 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 59 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 402 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 31 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 1460 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1273 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1272 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 126 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 26 exceeds gate value 25 for firefox browserify home mean backgroundConnect
Benchmark value 27 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 15 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 4 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 1251 exceeds gate value 1230 for firefox browserify home mean loadScripts
Benchmark value 14 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1778 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 1496 exceeds gate value 1495 for firefox browserify home p95 load
Benchmark value 333 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 39 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 19 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 40 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1750 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1490 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1490 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 110 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 44 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 31 exceeds gate value 15 for firefox webpack home mean getState
Benchmark value 9 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1470 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 16 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 2180 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1741 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1740 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 288 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 51 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 15 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1701 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 37 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 842ms | Sum of p95 exceeds: 1197.8ms
Sum of all benchmark exceeds: 2039.8ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: 17.1 KiB (0.32%)
  • ui: 15.7 KiB (0.21%)
  • common: -55.28 KiB (-0.64%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-swaps-and-bridge Swaps and Bridge team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants