Skip to content

Retire no_verify_by_mail_for_biometric_comparison_enabled feature flag#10977

Merged
jmhooper merged 2 commits intomainfrom
jmhooper-retire-no-gpo-feature-flag
Jul 24, 2024
Merged

Retire no_verify_by_mail_for_biometric_comparison_enabled feature flag#10977
jmhooper merged 2 commits intomainfrom
jmhooper-retire-no-gpo-feature-flag

Conversation

@jmhooper
Copy link
Contributor

The no_verify_by_mail_for_biometric_comparison_enabled was used to control whether the GPO option was available for users performing a biometric comparison. We have enabled the feature flag in production and will be moving forward with this feature. This commit retires the feature flag.

The `no_verify_by_mail_for_biometric_comparison_enabled` was used to control whether the GPO option was available for users performing a biometric comparison. We have enabled the feature flag in production and will be moving forward with this feature. This commit retires the feature flag.

[skip changelog]
@jmhooper jmhooper requested a review from a team July 23, 2024 16:07

visit sign_out_url
user.reload
context 'with an sp that allows in person proofing' do
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This spec used to test in-person and GPO in one pass. That is no longer possible so I broke GPO and in-person out into 2 examples.

@jmhooper jmhooper merged commit fdf864d into main Jul 24, 2024
@jmhooper jmhooper deleted the jmhooper-retire-no-gpo-feature-flag branch July 24, 2024 18:52
mitchellhenke pushed a commit that referenced this pull request Jul 31, 2024
…lag (#10977)

The `no_verify_by_mail_for_biometric_comparison_enabled` was used to control whether the GPO option was available for users performing a biometric comparison. We have enabled the feature flag in production and will be moving forward with this feature. This commit retires the feature flag.

[skip changelog]
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.

2 participants