fix: localized connect message#41417
Conversation
|
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. |
Builds ready [84a5732]
⚡ Performance Benchmarks (Total: 🟢 18 pass · 🟡 0 warn · 🔴 0 fail)
🌐 Dapp Page Load BenchmarksCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [430ea29]
⚡ Performance Benchmarks (Total: 🟢 16 pass · 🟡 2 warn · 🔴 0 fail)
🌐 Dapp Page Load BenchmarksCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs
|
|



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>mesageRelated issues
Fixes: https://consensyssoftware.atlassian.net/browse/MUL-1403?atlOrigin=eyJpIjoiNWM4YjJlMmIxZWNiNGU2MmEwZTA1N2ZiMjMzZTI0YWYiLCJwIjoiaiJ9
Manual testing steps
Looking for LedgerScreenshots/Recordings
No visual difference.
Pre-merge author checklist
Pre-merge reviewer checklist
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
connectHardwareaction 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.