Skip to content

Deploy RC 261 to Prod#7978

Merged
zachmargolis merged 13 commits intostages/prodfrom
stages/rc-2023-03-14
Mar 14, 2023
Merged

Deploy RC 261 to Prod#7978
zachmargolis merged 13 commits intostages/prodfrom
stages/rc-2023-03-14

Conversation

@zachmargolis
Copy link
Contributor

User-Facing Improvements

  • Accessibility: Improve personal key example description for assistive technology (#7965)
  • Identity Verification: Add guidance on how to edit Puerto Rico addresses for improved IdV success (#7964)
  • In-Person Proofing: Validate State ID and Address are transliterable to a valid USPS submission (#7938)
  • In-person proofing: Remove phone number from PO locations (#7953)

Internal

  • Code Quality: Remove unused code (#7957)
  • Database: Add primary database socket configuration (#7967)
  • Development: Skip loading yard gem by default (#7954)
  • IdV Fraud: Automatically reject profiles after 30 (#7940)
  • In-Person Proofing: Do not return HTTP 500 when cross-site request forgery token is invalid (#7950)
  • Logging: Add regression spec for IDV funnel database logs and log ssn step views (#7962)
  • Redis: Use one primary Redis connection pool (#7968)
  • Redis: Add configuration for IRS Attempts API Redis pool size (#7972)
  • Source code: Reorganize DocAuth::ProcessedAlertToLogAlertFormatter (#7959)

Upcoming Features

  • Fraud Mitigation: Implement spam protection for phone registration (#7952)

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.

🚀

zachmargolis and others added 13 commits March 14, 2023 10:23
**Why**: avoids mysterious segfaults on arm machines

changelog: Internal, Development, Skip loading yard gem by default
* create fraud rejection daily job

fraud_rejection_daily_job checks for profiles that have been in fraud
review for 30 days and then automatically rejects them

* add test for fraud rejection job

* added fraud job to cron job

* added ability to notify user

* add changelog

changelog: Internal, IdV Fraud, Automatically reject profiles after 30
days fraud review

* fix typo in notify

* add Fraud infront of analytics event

* add index to fraud_review_pending
* Remove unused personal key JavaScript behavior

changelog: Internal, Code Quality, Remove unused code

* Update yarn.lock
)

changelog: Internal, Source code, Reorganize DocAuth::ProcessedAlertToLogAlertFormatter
* Include parsed phone country code in reCAPTCHA logging

changelog: Upcoming Features, Fraud Mitigation, Implement spam protection for phone registration

* Qualify country code in reCAPTCHA result as from phone

* Avoid nil hash splatting
* Removing phone from barcode pages and search pages

* Undo JS refactor

* Remove phone field expectation from presenter spec

* Removing phone from proofer, spec and facilities config.

* Update mailer spec to remove phone

* changelog: User-facing Improvements, In-person proofing, remove phone number from PO locations

* remove unused translation keys

* undo mock change
…USPS submission (#7938)

* LG-7832: Validate State ID and Address are transliterable to a valid USPS submission

changelog: Improvements, In-Person Proofing, Validate State ID and Address are transliterable to a valid USPS submission

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

* LG-7832: Make test updates based on PR feedback

* Update spec/services/usps_in_person_proofing/enrollment_helper_spec.rb

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

* Update app/services/usps_in_person_proofing/enrollment_helper.rb

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

* LG-7832: Remove incorrect usage of .class

* LG-7832: Feature should only be enabled by default in dev environments

* Update app/services/idv/steps/in_person/address_step.rb

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

* Update app/services/idv/steps/in_person/state_id_step.rb

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

* LG-7832: Use case no longer requires support for HashWithIndifferentAccess

* LG-7832: Clean up translation usage

* LG-7832: Update test

* LG-7832: Update test

* LG-7832: Simplify unsupported character extraction from transliterated string

* LG-7832: Clarify transliteration result documentation

* LG-7832: Refactor helper logic into validator classes

* Switch to ->() lambda syntax to remove extra parens

* Change one more proc syntax to "stabby lambda"

* un-stub the TransliterableValidator spec as much as possible

- Real ActiveModel::Errors instance
- Model fields all defined in one spot
- spy on real validator.transliterator instead of stubbing

---------

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
Co-authored-by: Zach Margolis <zachary.margolis@gsa.gov>
Puerto Rico residents have a very low success rate with Lexisnexis Instant Verify. It appears that it expects the Urbanization part of the address to be on line 2. To gather data and take a first step toward helping Puerto Rico residents complete identity verification successfully, add guidance on how to edit their address.

changelog: User-facing Improvements, Identity Verification, Add guidance on how to edit Puerto Rico addresses for improved IdV success

* Change address entry field labels and fix newline in alert msg

* Show guidance only if state is PR, not if issuer is PR

* Add AddressPresenter to conditionally show edit field hints

* New address for Puerto Rico yml fixture

* Change AddressPresenter to take @pii

* Update banner text per discussion with Princess

* Replace a few explicit strings in specs with I18n.t lookups

* Make hint strings more readable
changelog: Internal, Database, Add primary database socket configuration
* LG-8894: Describe personal key example for assistive technology

changelog: User-Facing Improvements, Accessibility, Improve personal key example description for assistive technology

* Update personal key specs to test for code example
* Reduce to using one primary Redis connection pool

changelog: Internal, Redis, Use one primary Redis connection pool

* Update app/services/out_of_band_session_accessor.rb

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

* fix spec

* update gem tag version

---------

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
* LG-7832: Support multiple characters returned from transliteration; add testing around treatment of spaces

* [skip changelog]
changelog: Internal, Redis, Add configuration for IRS Attempts API Redis pool size
@zachmargolis zachmargolis force-pushed the stages/rc-2023-03-14 branch from a561529 to baafe05 Compare March 14, 2023 17:24
@zachmargolis zachmargolis merged commit 7385dae into stages/prod Mar 14, 2023
@zachmargolis zachmargolis deleted the stages/rc-2023-03-14 branch March 14, 2023 17:32
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.

6 participants