Skip to content

LG-13691: Standardize PIV/CAC terminology#10969

Merged
jmdembe merged 21 commits intomainfrom
jd/LG-13691-standardize-piv-cac
Aug 5, 2024
Merged

LG-13691: Standardize PIV/CAC terminology#10969
jmdembe merged 21 commits intomainfrom
jd/LG-13691-standardize-piv-cac

Conversation

@jmdembe
Copy link
Contributor

@jmdembe jmdembe commented Jul 19, 2024

🎫 Ticket

Link to the relevant ticket:
LG-13691

🛠 Summary of changes

This PR standardizes the use of "Government employee ID" and "PIV/CAC" throughout the application.

📜 Testing Plan

On account creation

  • Follow the steps to set up an account with a .gov email
  • After password creation, see the updated screen
English Spanish French Chinese
Screenshot of updated page in English Screenshot of updated page in Spanish Screenshot of updated page in French Screenshot of updated page in Chinese

When adding a PIV card

  • Follow the above directions to set up an account, then click "Add PIV/CAC"
  • See the updated screen
English Spanish French Chinese
Screenshot of updated Add your government employee id page in English Screenshot of add your government employee id page in Spanish screenshot of add your government id page in french screenshot of add your government id page in chinese

After the successful setup of a PIV card

  • Follow all of the above instructions to setup an account and add a PIV card
  • Once PIV card has successfully been added, see the updated screen
English Spanish French Chinese
screenshot of page confirming successful setup in english screenshot of page confirming successful setup in spanish screenshot of page confirming successful setup in french screenshot of page confirming successful setup in chinese

When signing in with a PIV/CAC

  • Click "Sign in with your government employee ID"
  • See updated screen
English Spanish French Chinese
screenshot of sign in with your government employee id screen in english screenshot of sign in with your government employee id screen in spanish screenshot of sign in with your government id screen in french screenshot of sign in with your government employee id screen in chinese

Error state: logging in with PIV not connected to an account

  • Follow above instructions, click "Insert PIV/CAC to continue"
  • Provide a PIV that has bot already been associated to an account. Click "Proceed
  • See updated screen
English Spanish French Chinese
screenshot of your government employee id is not connected to an account in english screenshot of your government id is not connected to an account in spanish screenshot of your government employee id is not connected to an account in french screenshot of your government employee id is not connected to an account in chinese

Error state: User tries to register PIV card, no certificate is available
Note: Can be done on an existing account or during account creation. The instructions below is for the account creation flow

  • With a .gov or .mil email address, follow steps to create an account
  • Click "Add PIV card" when suggested to add an account
  • Follow steps to add an account
  • On local environment, give the sample PIV card a name and select "No certificate"
  • See the updated screen
English Spanish French Chinese
Update of we cannot detect a certificate on your government employee screen in english Update of we cannot detect a certificate on your government employee screen in spanish Update of we cannot detect a certificate on your government employee screen in french Update of we cannot detect a certificate on your government employee screen in chinese

Sidebar on "Your account" page
Requirement: An existing account

  • Follow steps to create an account.
  • Successfully set up an MFA method
  • Arrive on your account page
  • On the left, see the updated content
English Spanish French Chinese
updated add your government employee id text in english updated add your government employee id text in spanish updated add your government employee id text in french updated add your government employee id text in chinese

Banner after PIV is added to account
Requirement: An existing account

  • Follow steps to create an account
  • On the right side of the Account page, click "Add your government employee ID
  • Follow steps to add a PIV to your account
  • See banner
English Spanish French Chinese
screenshot of a government employee was added to your account banner in english screenshot of a government employee id was added to your account banner in spanish screenshot of a government employee was added to your account banner in french screenshot of a government employee was added to your account banner in chinese

@jmdembe jmdembe marked this pull request as draft July 19, 2024 18:56
@jmdembe jmdembe changed the title DRAFT: Standardize PIV/CAC terminology lg-13691: Standardize PIV/CAC terminology Jul 22, 2024
@jmdembe jmdembe changed the title lg-13691: Standardize PIV/CAC terminology LG-13691: Standardize PIV/CAC terminology Jul 22, 2024
@jmdembe jmdembe force-pushed the jd/LG-13691-standardize-piv-cac branch from 78476db to 15ea50c Compare July 22, 2024 17:58
@jmdembe jmdembe marked this pull request as ready for review July 24, 2024 14:34
@jmdembe jmdembe requested a review from a team July 24, 2024 14:35
Copy link
Contributor

@kevinsmaster5 kevinsmaster5 left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@jmdembe jmdembe marked this pull request as draft July 25, 2024 12:41
@jmdembe
Copy link
Contributor Author

jmdembe commented Jul 25, 2024

Returning to a draft PR to address logic change for error steps (sign in vs. authentication) and cleanup

@jmdembe
Copy link
Contributor Author

jmdembe commented Jul 30, 2024

Updated screen shots as discussed in this acceptance thread

Error state: PIV is not associated with an account page with updated button

English Spanish French Chinese
screenshot of error page in english screenshot of error page in spanish screenshot of error page in french screenshot of error page in chinese

Add your PIV/CAC screen with updated button text

Header Header Header Header
screen shot of add your PIV card page in English screenshot of add your PIV card page in Spanish screenshot of add your piv card page in french screen shot of add your piv card page in chinese

Error stage page with updated button

English Spanish French Chinese
screenshot of no certificate error page in english screenshot of no certificate error page in spanish screen shot of no certificate error page in french screen shot of no certificate error page in chinese

@jmdembe jmdembe marked this pull request as ready for review July 30, 2024 21:14
@jmdembe jmdembe requested a review from a team July 30, 2024 21:32
Comment on lines 11 to 13
<div class="margin-top-4">
<%= link_to t('instructions.mfa.piv_cac.back_to_sign_in'), root_url, class: 'usa-button' %>
</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

While we're here, I think we should fix up the fact that this button is the incorrect size and spacing relative to the preceding paragraph. The designs linked to the ticket also have the button in the correct size.

Compare:

Your Screenshots Figma Reference Design
image image
Suggested change
<div class="margin-top-4">
<%= link_to t('instructions.mfa.piv_cac.back_to_sign_in'), root_url, class: 'usa-button' %>
</div>
<%= render ButtonComponent.new(
url: new_user_session_url,
big: true,
wide: true,
class: 'display-block margin-top-5'
).with_content(t('instructions.mfa.piv_cac.back_to_sign_in')) %>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it. I made a few additional changes to the suggestion-- I removed the display-block class and then took a bit off the margin-top of the button so that the distance between the paragraph and the button is at 40px.

<%= render ButtonComponent.new(
      url: new_user_session_url,
      big: true,
      wide: true,
      class: 'margin-top-3'
    ).with_content(t('instructions.mfa.piv_cac.back_to_sign_in')) %>

Comment on lines +11 to +13
<%= render ButtonComponent.new(
url: setup_piv_cac_url,
class: 'usa-button margin-top-2',
Copy link
Contributor

Choose a reason for hiding this comment

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

Same note here about using the correct button size (big & wide). Also, ButtonComponent handles adding the usa-button class, so we don't need to add it.

Suggested change
<%= render ButtonComponent.new(
url: setup_piv_cac_url,
class: 'usa-button margin-top-2',
<%= render ButtonComponent.new(
url: setup_piv_cac_url,
big: true,
wide: true,
class: 'margin-top-2',

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree with with removing usa-button, but based on the Figma, it looks like the width should be the width of the content.

Suggested change Figma
image image

Copy link
Contributor

Choose a reason for hiding this comment

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

I think that might be a mistake in the Figma designs, since these page layouts typically use the "wide" button variant. Could you get some clarification from UX people on the team?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in e517e40

jmdembe and others added 3 commits August 2, 2024 09:25
Co-authored-by: Andrew Duthie <1779930+aduth@users.noreply.github.com>
Co-authored-by: Andrew Duthie <1779930+aduth@users.noreply.github.com>
Co-authored-by: Andrew Duthie <1779930+aduth@users.noreply.github.com>
@jmdembe jmdembe force-pushed the jd/LG-13691-standardize-piv-cac branch from 28b0792 to 23d8cce Compare August 2, 2024 18:03
@jmdembe jmdembe merged commit 72832aa into main Aug 5, 2024
@jmdembe jmdembe deleted the jd/LG-13691-standardize-piv-cac branch August 5, 2024 15:17
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