Add CTA to let users know they can paste mnemonic phrase into box#2803
Open
leofelix077 wants to merge 2 commits into
Open
Add CTA to let users know they can paste mnemonic phrase into box#2803leofelix077 wants to merge 2 commits into
leofelix077 wants to merge 2 commits into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds user-facing guidance in the Recover Account flow to make the existing “paste full mnemonic into first input to auto-fill” behavior discoverable (Issue #2494).
Changes:
- Updates the Recover Account screen copy to include a second line indicating the mnemonic can be pasted into the first box.
- Adds the new CTA string to English and Portuguese locale files (and updates the existing mnemonic prompt key to include a trailing period).
- Updates Playwright e2e tests/helpers to assert the new CTA is visible on the import screen.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| extension/src/popup/views/RecoverAccount/index.tsx | Adds a second line of localized body text advertising the paste-to-autofill behavior. |
| extension/src/popup/locales/en/translation.json | Adds the new CTA string and updates the mnemonic prompt key/value punctuation. |
| extension/src/popup/locales/pt/translation.json | Adds the new CTA string and updates the mnemonic prompt key/value punctuation. |
| extension/e2e-tests/onboarding.test.ts | Asserts the new CTA text is visible across import-related onboarding flows. |
| extension/e2e-tests/helpers/onboarding.ts | Extends the shared import-flow helper to assert the CTA is present. |
| extension/e2e-tests/helpers/login.ts | Extends login helper’s import flow assertion to include the CTA. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
PR Preview build is ready: https://github.com/stellar/freighter/releases/tag/untagged-2594fa59cfa495d12582 (SDF collaborators only — install instructions in the release description) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #2494
Quick-fix: Adds a new line of text to let users know they can paste their phrase on the first box
Didn't change the first line text, as it was understood from the conversation that the intention was to only add the sub text, not the original first line