Skip to content

LG-9545 Help Center article about Personal Key#1109

Merged
theabrad merged 5 commits intomainfrom
abrad-lg-9545-pk-help
Jun 30, 2023
Merged

LG-9545 Help Center article about Personal Key#1109
theabrad merged 5 commits intomainfrom
abrad-lg-9545-pk-help

Conversation

@theabrad
Copy link
Contributor

🎫 Ticket

LG-9545

🛠 Summary of changes

Adding a help center article for Personal Key.

📸 Screenshots

Trouble Signing In
Screenshot 2023-06-28 at 10 30 16 AM

Manage Your Account:

EN:
Screenshot 2023-06-28 at 10 29 06 AM

ES:
Screenshot 2023-06-28 at 10 29 38 AM

FR:
Screenshot 2023-06-28 at 10 29 50 AM

permalink: /es/help/manage-your-account/personal-key/
order: 3
redirect_from:
- /es/es/help/manage-your-account/personal-key/
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this path have two /es/es in it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope.


### If you don’t have a personal key

If you reset your password without your personal key, you’ll need to verify your identity again.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it worth specifying here that if they change their password while logged in, this doesn't apply?

Comment on lines +46 to +48
[Forgot your password](/es/help/trouble-signing-in/forgot-your-password/)
[Change your password](/es/help/manage-your-account/change-your-password/)
[How to verify your identity](/es/help/verify-your-identity/how-to-verify-your-identity/)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should these link texts be in Spanish?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There weren't any translations in the doc and it was a question if translations should be included. I'll ask.

Copy link
Contributor

Choose a reason for hiding this comment

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

This is how the link shows up for me when I switch to the languages at the top dropdown, so I think the links are valid as they are. Spanish example: https://login.gov/es/help/trouble-signing-in/forgot-your-password/

Comment on lines +47 to +49
[Forgot your password](/fr/help/trouble-signing-in/forgot-your-password/)
[Change your password](/fr/help/manage-your-account/change-your-password/)
[How to verify your identity](/fr/help/verify-your-identity/how-to-verify-your-identity/)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should these link texts be in French?

Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above for Spanish

Comment on lines +3 to +4
title: Forgot your personal key
category: trouble-signing-in
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this supposed to be the same content (or very similar) as the set of pages above? Can we provide a link to the other page to avoid duplicating the content?

@aduth aduth requested a review from a team June 28, 2023 16:00
@artfulaction
Copy link
Contributor

artfulaction commented Jun 29, 2023

For the Spanish and French screenshots, I don't see the translations for "Personal key" link listed in the sidebar?

The English version looks good, so I'm thinking we just have to match the structure in the sidebar for the other languages.

title: Personal key
category: manage-your-account
permalink: /help/manage-your-account/personal-key/
order: 3
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we update the order values for the others in the list, since this conflicts with "Add or change your authentication method"

https://github.com/18F/identity-site/blob/fb4fdde626c84c4cb458490037a2586b9635eb3c/content/_help/manage-your-account/add-or-change-your-authentication-method._en.md#L6

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

@theabrad theabrad merged commit 50e1724 into main Jun 30, 2023
@theabrad theabrad deleted the abrad-lg-9545-pk-help branch June 30, 2023 21: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