Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LG-12805 verify your id in person #1234

Merged
merged 10 commits into from
Apr 15, 2024

Conversation

svalexander
Copy link
Contributor

@svalexander svalexander commented Apr 10, 2024

🎫 Ticket

Link to the relevant ticket.
LG-12805

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

@svalexander svalexander changed the title Shannon/lg 12805 verify your id in person LG-12805 verify your id in person Apr 10, 2024
Copy link
Contributor

@rutvigupta-design rutvigupta-design left a comment

Choose a reason for hiding this comment

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

@svalexander everything looks right to me except one tiny thing in the French translations. In the second section (under Commencez cette procédure sur Login.gov avant de vous rendre au bureau de poste), I think we're missing the closing "»" and there's an extra space after "de poste". Tried highlighting this but let me know if you have any questions! Noting that this is missing in the translations document!
Screen Shot 2024-04-11 at 2 15 17 PM

@svalexander
Copy link
Contributor Author

@rutvigupta-design should it look like this? « Vérifier votre identité dans un bureau de poste ».

@rutvigupta-design
Copy link
Contributor

@svalexander yes perfect, it should look like that!

@svalexander
Copy link
Contributor Author

@rutvigupta-design ok awesome, i just pushed up that change

Copy link
Contributor

@rutvigupta-design rutvigupta-design left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@svalexander svalexander force-pushed the shannon/lg-12805-verify-your-id-in-person branch from ee93d48 to 0961dd6 Compare April 15, 2024 14:41
@svalexander svalexander merged commit 76d68b9 into main Apr 15, 2024
14 checks passed
@svalexander svalexander deleted the shannon/lg-12805-verify-your-id-in-person branch April 15, 2024 14:51
mitchellhenke pushed a commit that referenced this pull request May 6, 2024
* add en strings for verify your id in person pg

* add es translations for verify your id in person pg

* add fr translations for verify your id in person pg

* update fr translation

* add space btwn paragraphs

* add another space btwn paragraphs

* Testing gpg signing

* Testing gpg signing again

* Testing gpg signing

* Testing gpg signing again
mitchellhenke pushed a commit that referenced this pull request May 6, 2024
* add en strings for verify your id in person pg

* add es translations for verify your id in person pg

* add fr translations for verify your id in person pg

* update fr translation

* add space btwn paragraphs

* add another space btwn paragraphs

* Testing gpg signing

* Testing gpg signing again

* Testing gpg signing

* Testing gpg signing again
mitchellhenke pushed a commit that referenced this pull request May 6, 2024
* add en strings for verify your id in person pg

* add es translations for verify your id in person pg

* add fr translations for verify your id in person pg

* update fr translation

* add space btwn paragraphs

* add another space btwn paragraphs

* Testing gpg signing

* Testing gpg signing again

* Testing gpg signing

* Testing gpg signing again
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