Skip to content

Deploy RC 222 to Prod#7231

Merged
zachmargolis merged 12 commits intostages/prodfrom
stages/rc-2022-10-27
Oct 27, 2022
Merged

Deploy RC 222 to Prod#7231
zachmargolis merged 12 commits intostages/prodfrom
stages/rc-2022-10-27

Conversation

@zachmargolis
Copy link
Contributor

@zachmargolis zachmargolis commented Oct 26, 2022

Improvements

  • In-Person Proofing: Uses correct error handling (#7199)
  • User experience: Add alert to Webauthn Platform setup page (#7195)

Internal

  • Inherited Proofing: We are Retrieving API call (#7210)
  • Logging: Document doc auth analytics events (#7191)
  • Performance: Reduce JavaScript bundle size in critical paths (#7188)

holytoastr and others added 11 commits October 25, 2022 13:07
* Adds proofing session

Why:
Need to track asyn proofing session to accurately track proofing process

changelog: Internal, Inherited Proofing, We are Retrieving API call

* Update app/services/idv/steps/inherited_proofing/agreement_step.rb

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

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

* Add alert component to webauthn platform setup page

* Move alert

* Add translations

* Add spec

changelog: Improvements, User experience, Add alert to Webauthn Platform setup page [LG-7745]

* Change spacing

* Change margin
The selfie is used to support strict IAL2. The strict IAL2 flow is being retired so we will not be matching selfies here in the near term. This commit retires the code that passes the selfie through the IdV agent to the document proofing job and to the SDK adapter since it is no longer an active code path.

[skip changelog]
* Remove liveness check from ApiDocumentVerification

[skip changelog]

* remove selfie_image_iv
)

* Remove DOC_AUTH constant
* Document actions
* Update tests to include analytics_id
* alphabetize
* Add inherited proofing stubs
* Add optional steps for inherited proofing
* Remove check for nil @analytics_ids so we always generate events (#7213)

changelog: Internal, Logging, Document doc auth analytics events
)

There was a feature flag to disable the letter check when strict IAL2 was requested. We are retiring IAL2 strict so the letter restriction and feature flag are no longer necessary. This commit removes both.

[skip changelog]
…#7219)

* Remove selfie and liveness from ApiImageUploadForm and relevant specs

* [skip changelog]

* Add comma to make linter happy
Removed IAL2 strict code from AuthorizationController and OpenIDConnectAuthorizeForm
* changelog: Improvements, In-Person Proofing, uses correct error handling

* changelog: Improvements, In-Person Proofing, adds method for ArcGIS API findAddressCandidates

* Add error states, comments, and renaming

* Update app/services/arcgis_api/geocoder.rb

Co-authored-by: Sheldon Bachstein <sheldon.bachstein@gsa.gov>

* Update app/services/arcgis_api/geocoder.rb

Co-authored-by: Sheldon Bachstein <sheldon.bachstein@gsa.gov>

* Update app/services/arcgis_api/geocoder.rb

Co-authored-by: Sheldon Bachstein <sheldon.bachstein@gsa.gov>

* Simplify test and location coordinates data structure

Co-authored-by: Sheldon Bachstein <sheldon.bachstein@gsa.gov>
* Implement language picker as design system accordion

changelog: Internal, Performance, Reduce JavaScript bundle size in critical paths

* Convert language picker to ViewComponent

* Add specs for LanguagePickerComponent

* Flatten markup by assigning class on component

* Use dig for property lookup

Safer for intermediate nil handling

See: #7188 (comment)
Co-Authored-By: Zach Margolis <zbmargolis@gmail.com>

* Increase focus offset for language picker links

Co-authored-by: Zach Margolis <zbmargolis@gmail.com>
…n the USPS polling job (#7212)

* rename shared examples for easier searching

* add testing for error response message

* pass the correct hash for bad request errors

* Get specs running clean. Minor cleanups

* Restructure error handling with more error types

* Begin logging status code for HTTP errors

* Resolve todos

* Get rid of unnecessary variable

* Fix broken proofer spec

* Update stub name to fix broken spec

* Move NewRelic reporting into handler methods

* Get rid of unnecessary safe-navigation op

* Remove weird status code fallback check

The behavior doesn't match the other Faraday behavior for other error
types so it feels weird to rely on it just for ParsingError cases

* Add disclaimer comment

Co-authored-by: Tomas Apodaca <thomas.apodaca@gsa.gov>
* Collapse language picker by default

changelog: Bug Fixes, Page Layout, Avoid flickering language picker appearance on page load

* Add spec for language picker accessible relation

* Add spec for language options

(cherry picked from commit 1b71195)
@aduth
Copy link
Contributor

aduth commented Oct 27, 2022

Bug fixes

  • Page Layout: Avoid flickering language picker appearance on page load (#7236)

We can leave this out of the release notes since the refactor that introduced the regression would have first appeared in this release anyways, so it was never an issue affecting users.

@zachmargolis
Copy link
Contributor Author

We can leave this out of the release notes since the refactor that introduced the regression would have first appeared in this release anyways, so it was never an issue affecting users.

it has been removed

@zachmargolis zachmargolis merged commit 6df0b0a into stages/prod Oct 27, 2022
@zachmargolis zachmargolis deleted the stages/rc-2022-10-27 branch October 27, 2022 16:27
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