Skip to content

fix(PostConnect): move strings to _locales file#1332

Merged
sidvishnoi merged 7 commits intomainfrom
post-connect-error-messages-i18n
Mar 24, 2026
Merged

fix(PostConnect): move strings to _locales file#1332
sidvishnoi merged 7 commits intomainfrom
post-connect-error-messages-i18n

Conversation

@sidvishnoi
Copy link
Copy Markdown
Member

Context

Part of #1238
Follow-up of #1327

Changes proposed in this pull request

Move all message strings to _locales.

@github-actions github-actions Bot added area: i18n area: pages/app area: pages Changes to any of extension's pages labels Mar 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 19, 2026

Extension builds preview

Name Link
Latest commit f57cdce
Latest job logs Run #23427616115
Chrome (1.37MB)Download
Firefox (1.37MB)Download
Safari (2.32MB)Download

Comment thread src/_locales/en/messages.json Outdated
Comment thread src/_locales/en/messages.json Outdated
Comment thread src/_locales/en/messages.json Outdated
Comment thread src/_locales/en/messages.json Outdated
Base automatically changed from post-connect-error-messages-overhaul to main March 19, 2026 15:14
Comment thread src/_locales/en/messages.json Outdated
Comment thread src/_locales/en/messages.json Outdated
"message": "Wallet reconnection failed"
},
"postConnect_reconnect_failure_other_msg": {
"message": "We couldn’t verify your wallet. Please ensure the public key is added to your wallet."
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we're going to tell people to do something out of the ordinary--find the key and add it to their wallet--we should try to tell them how.

If this message appears on or over the screen that shows their key, then that's a little less info we need to provide.

We couldn't verify your wallet. Please copy your public key and add it to your digital wallet account.

Something like that, perhaps.

Copy link
Copy Markdown
Member Author

@sidvishnoi sidvishnoi Mar 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this message appears on or over the screen that shows their key, then that's a little less info we need to provide.

Can't do that without substantial changes unfortunately.

But will update message that they need to copy it manually or try again. c2a1aef

@sidvishnoi sidvishnoi merged commit 2cc0b6e into main Mar 24, 2026
13 checks passed
@sidvishnoi sidvishnoi deleted the post-connect-error-messages-i18n branch March 24, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: i18n area: pages/app area: pages Changes to any of extension's pages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants