Skip to content

LG-12061: All step indicator steps are shown completed on Personal Key page#10302

Merged
solipet merged 3 commits intomainfrom
dprice-lg-12061-step-indicator-on-personal-key-screen
Mar 25, 2024
Merged

LG-12061: All step indicator steps are shown completed on Personal Key page#10302
solipet merged 3 commits intomainfrom
dprice-lg-12061-step-indicator-on-personal-key-screen

Conversation

@solipet
Copy link
Contributor

@solipet solipet commented Mar 25, 2024

🎫 Ticket

Link to the relevant ticket:
LG-12061

🛠 Summary of changes

In #10280 we reworded the secure_account step to "Re-enter your password" so it no longer makes sense to have the final step be current once the user has re-entered their password.

The PR adds a new current_step of ALL_STEPS_COMPLETED which can be used to force all steps to render as completed.

👀 Screenshots

Before: lg-12061-before
After: lg-12061-after

page

In #10280 we reworded the `secure_account` step to "Re-enter your password"
so it no longer makes sense to have the final step be current once the user
has re-entered their password.

The PR adds a new current_step of `ALL_STEPS_COMPLETED` which can be
used to force all steps to render as completed.

changelog: User-Facing Improvements, Identity verification, On personal key screen the step indicator shows all steps complete.
@solipet solipet requested review from a team and aduth March 25, 2024 17:24
Copy link
Contributor

@jmax-gsa jmax-gsa 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.

👍

solipet and others added 2 commits March 25, 2024 16:15
Co-authored-by: Andrew Duthie <1779930+aduth@users.noreply.github.com>
@solipet solipet merged commit 83d39c4 into main Mar 25, 2024
@solipet solipet deleted the dprice-lg-12061-step-indicator-on-personal-key-screen branch March 25, 2024 21:45
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