Skip to content

fix: localized connect message#41417

Merged
montelaidev merged 2 commits into
mainfrom
fix/mul-1403
Apr 1, 2026
Merged

fix: localized connect message#41417
montelaidev merged 2 commits into
mainfrom
fix/mul-1403

Conversation

@montelaidev

@montelaidev montelaidev commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Description

This PR fixes the to use a localized message for the loading text in the Connect Hardware flow.

Changelog

CHANGELOG entry: Localize Looking for your <device> mesage

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/MUL-1403?atlOrigin=eyJpIjoiNWM4YjJlMmIxZWNiNGU2MmEwZTA1N2ZiMjMzZTI0YWYiLCJwIjoiaiJ9

Manual testing steps

  1. Go to add hardware wallet
  2. Connect a ledger device
  3. See the loading message as Looking for Ledger

Screenshots/Recordings

No visual difference.

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.

Note

Low Risk
Low risk: only changes the loading indicator string for hardware wallet connection and adjusts unit tests and locale files; no business logic or security-sensitive flows are modified.

Overview
Uses a new i18n key (hardwareWalletLookingForDevice) to render the Connect Hardware loading indicator instead of a hardcoded English string.

Adds the corresponding locale entries (US/GB English) and updates connectHardware action tests to pass a translation function and assert the localized loading message.

Written by Cursor Bugbot for commit 430ea29. This will update automatically on new commits. Configure here.

@montelaidev montelaidev self-assigned this Apr 1, 2026
@montelaidev montelaidev added the team-accounts-framework Accounts team label Apr 1, 2026
@github-actions

github-actions Bot commented Apr 1, 2026

Copy link
Copy Markdown
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.

@github-actions github-actions Bot added the size-S label Apr 1, 2026
@metamaskbotv2

metamaskbotv2 Bot commented Apr 1, 2026

Copy link
Copy Markdown
Contributor
Builds ready [84a5732]
⚡ Performance Benchmarks (Total: 🟢 18 pass · 🟡 0 warn · 🔴 0 fail)

Baseline (latest main): e40e8c1 | Date: 2/20/58219 | Pipeline: 23850615575 | Baseline logs

Interaction Benchmarks
Benchmarkchrome-browserify
loadNewAccount🟢 [Show logs]
confirmTx🟢 [Show logs]
bridgeUserActions🟢 [Show logs]

📈 Results compared to the previous 5 runs on main

  • loadNewAccount/load_new_account: +17%
  • loadNewAccount/total: +17%
Startup Benchmarks
Benchmarkchrome-browserifychrome-webpackfirefox-browserifyfirefox-webpack
startupStandardHome🟢 [Show logs]🟢 [Show logs]🟢 [Show logs]🟢 [Show logs]
startupPowerUserHome🟢 [Show logs]🟢 [Show logs]🟢 [Show logs]🟢 [Show logs]

📈 Results compared to the previous 5 runs on main

  • startupStandardHome/initialActions: -29%
  • startupPowerUserHome/domInteractive: +14%
  • startupPowerUserHome/backgroundConnect: +24%
  • startupPowerUserHome/setupStore: +12%
  • startupPowerUserHome/numNetworkReqs: +48%
  • startupStandardHome/firstPaint: -23%
  • startupPowerUserHome/numNetworkReqs: -11%
  • startupStandardHome/initialActions: +67%
  • startupPowerUserHome/backgroundConnect: -12%
  • startupPowerUserHome/setupStore: -24%
  • startupStandardHome/initialActions: +43%
  • startupStandardHome/setupStore: -12%
  • startupPowerUserHome/setupStore: +26%
User Journey Benchmarks
Benchmarkchrome-browserify
onboardingImportWallet🟢 [Show logs]
onboardingNewWallet🟢 [Show logs]
assetDetails🟢 [Show logs]
solanaAssetDetails🟢 [Show logs]
importSrpHome🟢 [Show logs]
sendTransactions🟢 [Show logs]
swap🟢 [Show logs]

📈 Results compared to the previous 5 runs on main

  • onboardingImportWallet/doneButtonToHomeScreen: -75%
  • onboardingImportWallet/openAccountMenuToAccountListLoaded: +63%
  • onboardingImportWallet/total: -38%
  • onboardingNewWallet/agreeButtonToOnboardingSuccess: +26%
  • onboardingNewWallet/doneButtonToAssetList: -30%
  • onboardingNewWallet/total: -24%
  • assetDetails/assetClickToPriceChart: -39%
  • assetDetails/total: -39%
  • solanaAssetDetails/assetClickToPriceChart: -50%
  • solanaAssetDetails/total: -50%
  • importSrpHome/loginToHomeScreen: +17%
  • importSrpHome/openAccountMenuAfterLogin: -56%
  • importSrpHome/homeAfterImportWithNewWallet: -44%
  • importSrpHome/total: -39%
  • swap/openSwapPageFromHome: -87%
  • swap/fetchAndDisplaySwapQuotes: +30%
  • swap/total: +14%
🌐 Dapp Page Load Benchmarks

Current Commit: 84a5732 | Date: 4/1/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.04s (±38ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 734ms (±36ms) 🟢 | historical mean value: 729ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 86ms (±11ms) 🟢 | historical mean value: 86ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.04s 38ms 1.02s 1.34s 1.07s 1.34s
domContentLoaded 734ms 36ms 714ms 1.01s 761ms 1.01s
firstPaint 86ms 11ms 64ms 184ms 96ms 184ms
firstContentfulPaint 86ms 11ms 64ms 184ms 96ms 184ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -1.4 MiB (-20.49%)
  • ui: -164.04 KiB (-1.92%)
  • common: 940.74 KiB (8.1%)

@montelaidev montelaidev added this pull request to the merge queue Apr 1, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 1, 2026
@gantunesr gantunesr added this pull request to the merge queue Apr 1, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 1, 2026
@gantunesr gantunesr added this pull request to the merge queue Apr 1, 2026
@gantunesr gantunesr removed this pull request from the merge queue due to a manual request Apr 1, 2026
@metamaskbotv2

metamaskbotv2 Bot commented Apr 1, 2026

Copy link
Copy Markdown
Contributor
Builds ready [430ea29]
⚡ Performance Benchmarks (Total: 🟢 16 pass · 🟡 2 warn · 🔴 0 fail)

Baseline (latest main): 084a8a9 | Date: 9/20/58219 | Pipeline: 23864262048 | Baseline logs

Interaction Benchmarks
Benchmarkchrome-browserify
loadNewAccount
🟡 load_new_account
[Show logs]
confirmTx🟢 [Show logs]
bridgeUserActions🟢 [Show logs]

📈 Results compared to the previous 5 runs on main

  • loadNewAccount/load_new_account: +14%
  • loadNewAccount/total: +14%
Startup Benchmarks
Benchmarkchrome-browserifychrome-webpackfirefox-browserifyfirefox-webpack
startupStandardHome🟢 [Show logs]🟢 [Show logs]🟢 [Show logs]🟢 [Show logs]
startupPowerUserHome🟢 [Show logs]🟢 [Show logs]🟢 [Show logs]
🟡 loadScripts
[Show logs]

📈 Results compared to the previous 5 runs on main

  • startupStandardHome/initialActions: +43%
  • startupPowerUserHome/backgroundConnect: -32%
  • startupPowerUserHome/initialActions: -17%
  • startupStandardHome/domInteractive: -12%
  • startupStandardHome/setupStore: +13%
  • startupPowerUserHome/numNetworkReqs: +36%
  • startupStandardHome/domInteractive: +15%
  • startupStandardHome/initialActions: -38%
  • startupStandardHome/setupStore: -19%
  • startupPowerUserHome/domInteractive: +10%
  • startupPowerUserHome/backgroundConnect: -14%
  • startupPowerUserHome/setupStore: -13%
  • startupStandardHome/initialActions: -38%
  • startupStandardHome/setupStore: -29%
  • startupPowerUserHome/domInteractive: -23%
User Journey Benchmarks
Benchmarkchrome-browserify
onboardingImportWallet🟢 [Show logs]
onboardingNewWallet🟢 [Show logs]
assetDetails🟢 [Show logs]
solanaAssetDetails🟢 [Show logs]
importSrpHome🟢 [Show logs]
sendTransactions🟢 [Show logs]
swap🟢 [Show logs]

📈 Results compared to the previous 5 runs on main

  • onboardingImportWallet/metricsToWalletReadyScreen: +22%
  • onboardingImportWallet/doneButtonToHomeScreen: -76%
  • onboardingImportWallet/openAccountMenuToAccountListLoaded: +43%
  • onboardingImportWallet/total: -39%
  • onboardingNewWallet/skipBackupToMetricsScreen: -20%
  • onboardingNewWallet/agreeButtonToOnboardingSuccess: -23%
  • onboardingNewWallet/doneButtonToAssetList: -21%
  • onboardingNewWallet/total: -17%
  • assetDetails/assetClickToPriceChart: -24%
  • assetDetails/total: -24%
  • solanaAssetDetails/assetClickToPriceChart: -64%
  • solanaAssetDetails/total: -64%
  • importSrpHome/loginToHomeScreen: +22%
  • importSrpHome/homeAfterImportWithNewWallet: -43%
  • importSrpHome/total: -36%
  • swap/openSwapPageFromHome: -87%
  • swap/fetchAndDisplaySwapQuotes: +29%
🌐 Dapp Page Load Benchmarks

Current Commit: 430ea29 | Date: 4/1/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.03s (±69ms) 🟡 | historical mean value: 1.03s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 730ms (±68ms) 🟢 | historical mean value: 726ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 83ms (±10ms) 🟢 | historical mean value: 86ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.03s 69ms 992ms 1.30s 1.24s 1.30s
domContentLoaded 730ms 68ms 692ms 989ms 931ms 989ms
firstPaint 83ms 10ms 64ms 160ms 92ms 160ms
firstContentfulPaint 83ms 10ms 64ms 160ms 92ms 160ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs
  • background: 58 Bytes (0%)
  • ui: 5 Bytes (0%)
  • common: 144 Bytes (0%)

@sonarqubecloud

sonarqubecloud Bot commented Apr 1, 2026

Copy link
Copy Markdown

@gantunesr gantunesr added this pull request to the merge queue Apr 1, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 1, 2026
@montelaidev montelaidev added this pull request to the merge queue Apr 1, 2026
Merged via the queue into main with commit 0b94eec Apr 1, 2026
390 of 394 checks passed
@montelaidev montelaidev deleted the fix/mul-1403 branch April 1, 2026 23:34
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 1, 2026
@metamaskbot metamaskbot added the release-13.26.0 Issue or pull request that will be included in release 13.26.0 label Apr 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.26.0 Issue or pull request that will be included in release 13.26.0 size-S team-accounts-framework Accounts team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants