Skip to content

LG-13005 - use TransactionStatus to determine if document passed TrueID#12177

Merged
amirbey merged 45 commits intomainfrom
amirbey/LG-13005-transaction-status
May 19, 2025
Merged

LG-13005 - use TransactionStatus to determine if document passed TrueID#12177
amirbey merged 45 commits intomainfrom
amirbey/LG-13005-transaction-status

Conversation

@amirbey
Copy link
Contributor

@amirbey amirbey commented May 14, 2025

🎫 Ticket

Link to the relevant ticket:
LG-13005

🛠 Summary of changes

Use TrueIdResponse transaction status value (passed/failed) to determine if the document was successfully authenticated.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • DocAuth succeeds after submitting a yml document which includes transaction_status: passed
  • DocAuth fails after submitting a yml document which includes transaction_status: failed
  • DocAuth succeeds after submitting a yml document with transaction_status: passed and 2D Barcode Read error after user confirms name and DOB

amirbey added 30 commits May 14, 2025 08:54
changelog: Internal, Document Authentication, TrueIDReponse successful if transaction status passes
@amirbey amirbey self-assigned this May 14, 2025
@amirbey amirbey changed the title LG-13005 - TrueID transactions status LG-13005 - use TransactionStatus to determine if document passed TrueID May 14, 2025
@amirbey amirbey marked this pull request as ready for review May 14, 2025 18:38
Copy link
Contributor

@AShukla-GSA AShukla-GSA left a comment

Choose a reason for hiding this comment

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

Tested with 3 ymls locally, LGTM!

@amirbey amirbey merged commit 4d166de into main May 19, 2025
1 check passed
@amirbey amirbey deleted the amirbey/LG-13005-transaction-status branch May 19, 2025 19:06
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