Skip to content

Deploy RC 401 to Production#11002

Merged
zachmargolis merged 18 commits intostages/prodfrom
stages/rc-2024-07-30
Jul 30, 2024
Merged

Deploy RC 401 to Production#11002
zachmargolis merged 18 commits intostages/prodfrom
stages/rc-2024-07-30

Conversation

@zachmargolis
Copy link
Contributor

User-Facing Improvements

  • Doc Auth: Failed doc auth offers IPP offramp (#10903)

Bug Fixes

  • Accessibility: Use aria-hidden for decorative SVG images (#10986)

Internal

  • Automated Testing: Improve performance of PIV IdV sign-in spec (#10967)
  • Automated Testing: Fix flakey IAA Agreements spec (#10989)
  • CI: Change DNS zone for review apps to reviewapps (#10961)
  • Code Quality: Sort keys in application.yml.default (#10981)
  • Components: Use ActiveModel validations for components (#10971)
  • In-person proofing: Set the sponsor_id field on (#10984)
  • In-person proofing: Add enhanced_ipp property to (#10962)
  • Maintenance: Remove unused methods (#11000)
  • Source code: Add and fix predicate naming lint (#10990)

Upcoming Features

  • Authentication: Recaptcha Sign in (#10944)
  • Authentication: Let users who's passwords are compromised to change their password (#10861)
  • Enhanced In-Person Proofing: Content Updates For Ready to Verify View and Email for EIPP (#10974)
  • IdV with Biometric Comparison: Adding acr_values (#10993)
  • In-person proofing: Backfill sponsor id again (#10988)

theabrad and others added 18 commits July 24, 2024 14:05
* Failed doc auth result offers IPP offramp

changelog: User-Facing Improvements, Doc Auth, Failed doc auth offers IPP offramp

* remove showAlternativeProofingOptions

since we offer IPP offramp for failed doc_auth we do not need this check anymore
…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]
* normalize-yaml: Add option to ignore keys in sorting

* Sort keys in application.yml.default

changelog: Internal, Code Quality, Sort keys in application.yml.default

* Clarify ignore key sort option in README

* Add spec for ignoreKeySort

* Improve formatting for original and expected output

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>

* Fix comment annotation arrangement

---------

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
)

* Update documentation in all languages

* Rename language resouces

* changelog: Upcoming Features, Enhanced In-Person Proofing, Content Updates For Ready to Verify View and Email for EIPP
…10984)

changelog: Internal, In-person proofing, Set the sponsor_id field on
in-person enrollments when they are created during the IPP usps location
controller update request.
* Stub proofed user for PIV IdV sign-in spec

changelog: Internal, Automated Testing, Improve performance of PIV IdV sign-in spec

* Update spec references to OIDC Continue button
changelog: Internal, In-person proofing, Add enhanced_ipp property to
the event properties of the "GetUspsProofingResultsJob: Enrollment
status updated" analytic event.
* Use aria-hidden for decorative SVG images

changelog: Bug Fixes, Accessibility, Use aria-hidden for decorative SVG images

* Fix error on missing key method

* Use positive check for selecting non-aria-hidden img

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>

* Add explainer to accessibility matcher

* Update accessibility matcher name to match  new behavior

---------

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
changelog: Internal, Automated Testing, Fix flakey IAA Agreements spec
* Use ActiveModel validations for components

changelog: Internal, Components, Use ActiveModel validations for components

* Avoid unnecessary predicate method for loop
changelog: Upcoming Features, IdV with Biometric Comparison, Adding acr_values
---------

Co-authored-by: Lauren George <lauren.george@gsa.gov>
* changelog: Upcoming Features, Authentication, Recaptcha Sign in

* fix spec

* add test to ensure user redirected to security check failed

* add additional languges and page

* address comments

* lint fix

* address comments

* use c

* fix failed

* sign in security check controller spec added

* change spec name

* use back instead of cancel

* update security check failed spec
…onments (#10988)

* restore rake task to backfill sponsor_id column in in_person_enrollments table

* Changelog: Upcoming Features, In-person proofing, backfill sponsor id again

* update in_person_enrollments factory and get tests passing

* do clean up
* changelog: Upcoming Features, Authentication, let users who's passwords are compromised to change their password

* add password compromised spec and edit objects

* sign in spec updated

* change to use user_password_params

* address comments

* change up to use password controller

* password compromised updates

* fix lint and use global variable

* update edit spec and global variable

* fix the sign in spec

* fix form spec

* address comments, update test to better redirect

* remove former check

* make sure invalid password method is correct

* Update to make aria described conditional

* update specs and use hash format for attributes instead of positional arguments

* fix reset password form spec

* remove conditional for password strength description
changelog: Internal, Source code, Add and fix predicate naming lint

* Add back prefix for has_gov_or_mil_email?

* Allow has_ prefix for predicate names


Co-authored-by: Andrew Duthie <1779930+aduth@users.noreply.github.com>

* Clean up other request method name

* Remove unused delegator on TwoFactorOptionsPresenter

- Old method name: user_has_gov_or_mil_email?

* Bring back a has_

---------

Co-authored-by: Andrew Duthie <1779930+aduth@users.noreply.github.com>
changelog: Internal, Maintenance, Remove unused methods
* changelog: Internal, CI, Change DNS zone for review apps to reviewapps
@zachmargolis zachmargolis requested a review from Sgtpluck July 30, 2024 15:26
@zachmargolis zachmargolis merged commit 9190a01 into stages/prod Jul 30, 2024
@zachmargolis zachmargolis deleted the stages/rc-2024-07-30 branch July 30, 2024 15:57
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.