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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

LG-12624 additional content update for getting started screen #10352

Merged

Conversation

jmax-gsa
Copy link
Contributor

@jmax-gsa jmax-gsa commented Apr 2, 2024

馃帿 Ticket

Link to the relevant ticket:
LG-12624

馃洜 Summary of changes

Updated sub-heading text for step-up users.
Updated bullet points 1 and 3, and their associated text.

馃摐 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Bring up the idp app and the sinatra sample sp oidc app
  • From the sample sp app, select 'identity verified' and log in as a new user.
  • Verify that the text on the IdV welcome screen is correct in English, Spanish, and French versions.
  • Proceed through IdV, and log out.
  • From the sample sp app, select 'biometric required' and log in as the same user.
  • Verify that the text on the IdV welcome screen is correct in English, Spanish, and French versions.

馃憖 Screenshots

If relevant, include a screenshot or screen capture of the changes.

New User (English):

new-user-en

New User (Spanish):

Screenshot 2024-04-10 at 1 12 41鈥疨M

New User (French):

Screenshot 2024-04-10 at 2 13 00鈥疨M

Step-up User (English):

step-up-en

Step-up User (Spanish):

Screenshot 2024-04-10 at 1 16 34鈥疨M

Step-up User (French):

Screenshot 2024-04-10 at 2 15 29鈥疨M

Updated screenshots 4/15 per LQA feedback

French "Getting Started" screen (new user)

Screenshot 2024-04-16 at 9 29 15鈥疉M

French "Getting Started" screen (step-up user)

Screenshot 2024-04-16 at 9 35 56鈥疉M

Spanish "Getting Started" screen (new user)

Screenshot 2024-04-16 at 9 31 32鈥疉M

Spanish "Getting Started" screen (step-up user)

Screenshot 2024-04-16 at 9 36 33鈥疉M

@jmax-gsa jmax-gsa marked this pull request as ready for review April 2, 2024 20:44
@jmax-gsa jmax-gsa requested a review from a team April 2, 2024 20:44
@jmax-gsa jmax-gsa force-pushed the jmax/LG-12624-additional-content-update-for-getting-started branch from cf5dc3a to 364fa5b Compare April 2, 2024 21:07
changelog: User-Facing Improvements, Biometrics, Improved language on getting started page.
Removed period from bullet point 1
@jmax-gsa jmax-gsa force-pushed the jmax/LG-12624-additional-content-update-for-getting-started branch from 0ed0762 to fba5fab Compare April 4, 2024 13:48
Copy link
Member

@matthinz matthinz left a comment

Choose a reason for hiding this comment

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

LGTM

@jmax-gsa jmax-gsa merged commit c90f722 into main Apr 16, 2024
2 checks passed
@jmax-gsa jmax-gsa deleted the jmax/LG-12624-additional-content-update-for-getting-started branch April 16, 2024 14:26
@mitchellhenke mitchellhenke mentioned this pull request Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants