Skip to content

Jmax/lg 9084 remove personal key in gpo feature flag#7948

Merged
jmax-gsa merged 7 commits intomainfrom
jmax/LG-9084-remove-personal-key-in-gpo-feature-flag
Mar 15, 2023
Merged

Jmax/lg 9084 remove personal key in gpo feature flag#7948
jmax-gsa merged 7 commits intomainfrom
jmax/LG-9084-remove-personal-key-in-gpo-feature-flag

Conversation

@jmax-gsa
Copy link
Contributor

@jmax-gsa jmax-gsa commented Mar 8, 2023

🎫 Ticket

LG-9084

🛠 Summary of changes

Removed the gpo_personal_key_after_otp feature flag. Updated code and specs accordingly.

Testing Plan

  • Set the gpo_personal_key_after_otp feature flag to false in your application.yml file.
  • Walk through GPO verification, and ensure that you get to the personal key step after GPO verification.

@jmax-gsa jmax-gsa requested a review from theabrad March 8, 2023 17:07
@jmax-gsa jmax-gsa self-assigned this Mar 8, 2023
Copy link
Contributor

@zachmargolis zachmargolis 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

@theabrad theabrad left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
Refactored step_indicator_concern spec in the process.
@jmax-gsa jmax-gsa requested a review from aduth March 9, 2023 16:36
@jmax-gsa jmax-gsa merged commit f4a5868 into main Mar 15, 2023
@jmax-gsa jmax-gsa deleted the jmax/LG-9084-remove-personal-key-in-gpo-feature-flag branch March 15, 2023 14:59
svalexander pushed a commit that referenced this pull request Mar 16, 2023
Removed feature flag and related code.

Refactored step_indicator_concern spec in the process.

[skip changelog]

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
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.

5 participants