Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More renaming to Facial Matching #11304

Merged
merged 5 commits into from
Oct 2, 2024
Merged

More renaming to Facial Matching #11304

merged 5 commits into from
Oct 2, 2024

Conversation

Sgtpluck
Copy link
Contributor

@Sgtpluck Sgtpluck commented Oct 2, 2024

🎫 Ticket

https://gitlab.login.gov/lg-people/Melba/backlog-fy24/-/issues/116

🛠 Summary of changes

This change wraps up the rest of the language change from biometric_comparison to facial_match.

  • fixes the OIDC test controller that is used for facial match testing
  • updates another private method that somehow avoided discovery the first go around
  • Updates the yml/erb files
  • Updates the analytics_events field descriptions

@@ -84,7 +84,7 @@ def acr_values(ial:, aal:)
'' => 'http://idmanagement.gov/ns/assurance/ial/1',
'1' => 'http://idmanagement.gov/ns/assurance/ial/1',
'2' => 'http://idmanagement.gov/ns/assurance/ial/2',
'biometric-comparison-required' => 'http://idmanagement.gov/ns/assurance/ial/2',
'facial-match-required' => 'http://idmanagement.gov/ns/assurance/ial/2',
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 value doesn't matter because the ial2_params method adds the values. (it currently adds a vector of trust, but i will be updating that soon)

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

I would have titled this something like "More renaming to Facial Matching" because I feel like "Update YML files" undersells it a bit.

The code LGTM!

@Sgtpluck Sgtpluck changed the title Update yml files More renaming to Facial Matching Oct 2, 2024
@Sgtpluck Sgtpluck merged commit 9ceb21a into main Oct 2, 2024
2 checks passed
@Sgtpluck Sgtpluck deleted the dmm/update-yml-files branch October 2, 2024 23:51
MrNagoo pushed a commit that referenced this pull request Oct 3, 2024
* changelog: Internal, Facial Match, Update language
colter-nattrass pushed a commit that referenced this pull request Oct 23, 2024
* changelog: Internal, Facial Match, Update language
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.

3 participants