Skip to content

Deploy RC 348 to Production#9942

Merged
matthinz merged 14 commits intostages/prodfrom
stages/rc-2024-01-18
Jan 18, 2024
Merged

Deploy RC 348 to Production#9942
matthinz merged 14 commits intostages/prodfrom
stages/rc-2024-01-18

Conversation

@matthinz
Copy link
Contributor

@matthinz matthinz commented Jan 18, 2024

User-Facing Improvements

  • Doc Auth: Remove code for not-ready survey. (#9856)
  • LookBook Component: Add description to Login Button component (#9934)

Bug Fixes

  • LookBook: Fix bug with empty param (#9938)

Internal

  • Continuous Integration: Remove temporary continuous integration check on content changes (#9931)
  • Dependencies: Update dependencies (#9936)
  • Doc Auth: Document capture session result attributes doc_auth_success and selfie_success. (#9890)
  • IdV Tests: Fix flaky verify info spec test (#9921)
  • Reporting: Remove unused IRS Weekly Summary Report (#9925)
  • Ruby: Upgrade to Ruby 3.3.0 (#9935)

Upcoming Features

  • Document Authentication: Show user hints during selfie capture (#9901)

matthinz and others added 14 commits January 16, 2024 09:14
* Initial draft of create-deploy-pr workflow

This Github Actions workflow can be triggered manually and will create a new PR for a deployment.

changelog: Internal, Deployment, Add Github Actions workflow to create deployment PRs.

* Updates to create-deploy-pr

- Add command line args, incl. --dry-run
- Refactor some stuff out to funcs
- Check for GH_TOKEN variable
- Push SHA to remote directly rather than checking out local branch

* Create release when deploy PR is merged

* Support patch deploys

* Update run-name for create-release

* Move check for RC branch earlier

Also technically make the check actually work

* Only return merged PRs in create-release

* Create release even if deploy pr is not most recent

* Error out if PR not found when creating release

* Remove unused command line args read
…doc` (#9913)

* Stop merging things that aren't PII from the document into `pii_from_doc`

The `Idv::Session#pii_from_doc` method has a name that suggests it returns the PII that is read from the document. Prior to this commit this was not the case because we merge data that was not read from the document into the `pii_from_doc` hash. This was done as a convienence so it is available when communicating to vendors in downstream steps.

This commit removes the merge and provides the additional data explicitly in downsteam step.

[skip changelog]
…9930)

Remove app/views/idv/in_person/ssn/show.html.erb, no longer used.

[skip changelog]
* LG-12036:  New DocumentCaptureSessionResult attributes to separately indicates result for document check and liveness check.

changelog: Internal, Doc Auth, Document capture session result attributes doc_auth_success and selfie_success.

* LG-12036: documentation and test.

* LG-12036: documentation and test.

* LG-12036: mock client response, waiting on changes from another card.

* LG-12036: update mock client response.

* LG-12036: documentation change.

* LG-12036: save the attributes when failure encountered.

* LG-12036: address comment.

* LG-12036: address comment.
changelog: Internal, Continuous Integration, Remove temporary continuous integration check on content changes
changelog: Internal, Reporting, Remove unused IRS Weekly Summary Report
* init display hint text on selfie

* show error hint for selfie capture

changelog: Upcoming Features, Document Authentication, Show user hints during selfie capture

* attempt to display hint text in acuant capture canvas component

* rendering selfie hint text

* hint text is fixed on the screen

Co-authored-by: Charles Ferguson <charles.ferguson@gsa.gov>

* type onImageCaptureFeedback function

* remove hardcode of selfie required

* fix spec for hint text

* happy linting

* default imageCaptureText to empty string

* remove quotes from i18n keys for selfie hints

* Update app/javascript/packages/document-capture/components/acuant-capture.scss

Co-authored-by: Charley Ferguson <charleyferguson@navapbc.com>

* happy linting

---------

Co-authored-by: Charles Ferguson <charles.ferguson@gsa.gov>
Co-authored-by: Charley Ferguson <charleyferguson@navapbc.com>
* remove before action

* move time to 2 seconds to fix flaky tests

there is a disconnect with redis and the rate limiter causing the tests
to be flaky if you use a 1 second difference.

I also changed the verify_info_step spec to check if the user is not
rate limited since it would have expired.

* add changelog

changelog: Internal, IdV Tests, Fix flaky verify info spec test

* remove be_limited check

* Bump time travel to 3 to see if that makes CI happy.

* Debug prints for CI

* More CI fiddling.

* So much CI twiddling.

* More, even.

* Try forcing reload from Redis

* Expire rate limiter manually

---------

Co-authored-by: John Maxwell <john.maxwell@gsa.gov>
* LG-11985:  remove not ready section.

changelog:  User-facing Improvement, Doc Auth, Remove code for not-ready survey.

* LG-11985: cleanup not ready translations etc.

* LG-11985: linter.
* Upgrade to Ruby 3.3.0

changelog: Internal, Ruby, Upgrade to Ruby 3.3.0

* update more gems

* new image sha
* adding descriptions since this component is partner facing

* changelog: User-Facing Improvements, LookBook Component, Add description to Login Button component
- Via: https://www.ruby-lang.org/en/news/2023/12/25/ruby-3-3-0-released/

* Update pry-doc to hide warning

changelog: Internal, Dependencies, Update dependencies
* remove empty option because it creates a stack trace

* Bug Fixes,LookBook,fix bug with empty param

* changelog:Bug Fixes,LookBook,fix bug with empty param
@matthinz matthinz requested a review from a team January 18, 2024 18:22
@matthinz
Copy link
Contributor Author

(Note that the Github Actions present in 9f510e9 are not currently active)

@matthinz matthinz merged commit e2143c9 into stages/prod Jan 18, 2024
@matthinz matthinz deleted the stages/rc-2024-01-18 branch January 18, 2024 20:20
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.

9 participants