Skip to content

changelog: content, backup codes, LG-6357#6400

Merged
SammySteiner merged 1 commit intomainfrom
LG-6357-backup-code-content
May 23, 2022
Merged

changelog: content, backup codes, LG-6357#6400
SammySteiner merged 1 commit intomainfrom
LG-6357-backup-code-content

Conversation

@SammySteiner
Copy link
Contributor

@SammySteiner SammySteiner commented May 23, 2022

Ticket for change is here: https://cm-jira.usa.gov/browse/LG-6357
Figma for change is here: https://www.figma.com/file/wIOL0ZP4aZvMUjKKuCG2u2/LG-6344-Backup-codes-copy-changes?node-id=0%3A1
Translations are from Gengo.

Running locally:
Screen Shot 2022-05-23 at 2 34 52 PM

Deployed to https://idp.ssteiner.identitysandbox.gov/ for testing.

@SammySteiner SammySteiner requested a review from a team May 23, 2022 18:33
clave de seguridad.
- Le otorgaremos 10 códigos que puede descargar, imprimir, copiar o
escribir. Ingresará un código cada vez que inicie sesión.
are_you_sure_title: '¿Está seguro de que desea usar códigos de respaldo?'
Copy link
Contributor

Choose a reason for hiding this comment

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

Spanish appears to be the only language with single quotes around the text, maybe the quotes should be removed?

Copy link
Contributor

Choose a reason for hiding this comment

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

That is, for the key are_you_sure_title

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good question. That field was in quotes before, as are many in the es yml. I assumed it was because of the special characters 🤷

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe for Spanish text, any kind of punctuation has to be escaped because it does not render the text as expected. I ran into this one, but I do not remember where...

@jmhooper
Copy link
Contributor

When we squash and merge this can we make sure to give it a descriptive title and body to help people going through the log or blame in the future some context for this change?

@SammySteiner SammySteiner merged commit 47a9fd6 into main May 23, 2022
@SammySteiner SammySteiner deleted the LG-6357-backup-code-content branch May 23, 2022 19:46
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.

4 participants