Skip to content

LG-10511: Update copy for banner that notifies user they already tried to verify with a phone number#8911

Merged
amirbey merged 4 commits intomainfrom
amirbey/LG-10511-phone-failed
Aug 1, 2023
Merged

LG-10511: Update copy for banner that notifies user they already tried to verify with a phone number#8911
amirbey merged 4 commits intomainfrom
amirbey/LG-10511-phone-failed

Conversation

@amirbey
Copy link
Contributor

@amirbey amirbey commented Aug 1, 2023

🎫 Ticket

LG-10511

🛠 Summary of changes

Update copy for alert that notifies users they are retrying a failed number during IDV phone step.

📜 Testing Plan

  • Submit a number that fails phone verification step
  • Try again and type the same number into the phone textfield

👀 Screenshots

Before: LG-9499-gpo-unavailable LG-9499-fra LG-9499-es
After: LG-10511-en LG-10511-fr LG-10511-es LG-10511-fr-instead LG-10511-es-instead LG-10511-en-instead

changelog: User-Facing Improvements, Identity Verification, update copy for alert shown when user retries failed phone number
@amirbey amirbey changed the title update copy for retrying failed number alert LG-10511: Update copy for banner that notifies user they already tried to verify with a phone number Aug 1, 2023
@amirbey amirbey self-assigned this Aug 1, 2023
@amirbey amirbey requested a review from a team August 1, 2023 14:30
@amirbey amirbey marked this pull request as ready for review August 1, 2023 14:39
gpo_alert_html: Si vous n’avez pas <strong>d’autre</strong> numéro de téléphone
à essayer, %{link_html}.
gpo_alert_html: Essayez un <strong>autre</strong> numéro ou %{link_html}.
gpo_verify_link: vérifiez plutôt par courrier
Copy link
Contributor

Choose a reason for hiding this comment

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

The other translations with this tag just say "verify by mail". Do we want to update this one to match? It says "verify by mail instead".

Suggested change
gpo_verify_link: vérifiez plutôt par courrier
gpo_verify_link: vérifiez par courrier

Copy link
Contributor Author

@amirbey amirbey Aug 1, 2023

Choose a reason for hiding this comment

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

English also says "instead" but it's not part of the link. However, Spanish does not. Because of the way the words fall between translations, "instead" can be de-grouped from the link in English.

@artfulaction - thoughts? 🤔

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's remove "instead" from all (including English) to keep it consistent. Thanks for catching that!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

instead removed:

LG-10511-fr-instead LG-10511-es-instead LG-10511-en-instead

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good to me!

Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

LGTM!

@amirbey amirbey merged commit 0449ebb into main Aug 1, 2023
@amirbey amirbey deleted the amirbey/LG-10511-phone-failed branch August 1, 2023 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants