Skip to content

Update documentation and yml files for selfie#425

Merged
night-jellyfish merged 5 commits intomainfrom
brittany/update-yml-docs-for-selfie-data
Jan 11, 2024
Merged

Update documentation and yml files for selfie#425
night-jellyfish merged 5 commits intomainfrom
brittany/update-yml-docs-for-selfie-data

Conversation

@night-jellyfish
Copy link
Contributor

@night-jellyfish night-jellyfish commented Jan 10, 2024

This change:

  • updates the sample_full_error.yml for selfie entries, with the possible values notated
  • updates the text in the yml section to include selfie
  • removes a note that I don't think is true anymore (Even if you put all passing information into the test yaml file it will still produce an error. - you can upload a yml file that passes doc auth)
  • updates the yml docs to include a note that only the back image is evaluated
  • updates the general documentation for a note about changing yml files since I had trouble finding a note about it

Note: These changes should not be merged until 18F/identity-idp#9882 is merged.

Co-authored-by: Charley Ferguson <charleyferguson@navapbc.com>
Brittany Greaner added 2 commits January 10, 2024 13:58
Need to update the interface here to match a change requested on the IDP PR. Also updating the comment to clarify the difference between the meaning of the values.
@night-jellyfish
Copy link
Contributor Author

18F/identity-idp#9882 has now been merged so this can be merged when ready.

Copy link
Contributor

@charleyf charleyf left a comment

Choose a reason for hiding this comment

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

Text changes look good to me.

@night-jellyfish night-jellyfish merged commit 6a99795 into main Jan 11, 2024
@night-jellyfish night-jellyfish deleted the brittany/update-yml-docs-for-selfie-data branch January 11, 2024 21:30
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.

2 participants