Skip to content

LG-15700 - Update mock proofer to handle passport YAML#12071

Merged
jmax-gsa merged 5 commits intomainfrom
jmax/LG-15700-update-mock-proofer-to-handle-passports
Apr 16, 2025
Merged

LG-15700 - Update mock proofer to handle passport YAML#12071
jmax-gsa merged 5 commits intomainfrom
jmax/LG-15700-update-mock-proofer-to-handle-passports

Conversation

@jmax-gsa
Copy link
Contributor

@jmax-gsa jmax-gsa commented Apr 14, 2025

🎫 Ticket

Link to the relevant ticket:
LG-15700

🛠 Summary of changes

Write a brief description of what you changed.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Running locally, create an account and enter verification.
  • Select "upload images from this computer", and upload the file passport_credential.yml from the spec/fixtures directory.
  • Verify that your passport credentials are accepted, and you're on the SSN page
  • Cancel IdV and start over. Repeat the above procedure, but using the file passport_bad_mrz_credential.yml
  • Verify that your passport credentials were not accepted. and you're on the 'try again' page.

changelog: Internal, Testing, Added support for passports to the Mock Proofer.
Copy link
Contributor

@lmgeorge lmgeorge left a comment

Choose a reason for hiding this comment

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

I only have one change for documentation but otherwise LGTM. (caveat I did a read-only review for the sake of time)

@jmax-gsa jmax-gsa merged commit 9a538e9 into main Apr 16, 2025
1 check passed
@jmax-gsa jmax-gsa deleted the jmax/LG-15700-update-mock-proofer-to-handle-passports branch April 16, 2025 17:24
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