Skip to content

LG-9085 Add address_line2_present to verify proofing results analytics#7939

Merged
soniaconnolly merged 2 commits intomainfrom
sonia-lg-9085-log-line2-with-proofing
Mar 7, 2023
Merged

LG-9085 Add address_line2_present to verify proofing results analytics#7939
soniaconnolly merged 2 commits intomainfrom
sonia-lg-9085-log-line2-with-proofing

Conversation

@soniaconnolly
Copy link
Contributor

🎫 Ticket

LG-9085

🛠 Summary of changes

Add address_line2_present to IdV: doc auth verify proofing results analytics event when verify proofing completes. This will give us more data about how address line 2 is affecting proofing outcomes, and can be correlated with address_line2_present in IdV: doc auth image upload vendor submitted analytics event which shows whether address line2 was received from the vendor earlier in the flow.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Create account
  • Proof identity without address line 2
  • Check logs for doc auth verify proofing results event and presence of address_line2_present: false
  • Create another account
  • Proof identity with address line 2
  • Check logs for doc auth verify proofing results event and presence of address_line2_present: true

changelog: Internal, Identity Verification, improved address line 2 logging
@soniaconnolly soniaconnolly requested review from a team and matthinz March 7, 2023 01:17
Copy link
Contributor

@jmhooper jmhooper left a comment

Choose a reason for hiding this comment

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

Looks like the analytics spec needs some work but once that is done I think this is good to go.

@soniaconnolly soniaconnolly merged commit b0d263d into main Mar 7, 2023
@soniaconnolly soniaconnolly deleted the sonia-lg-9085-log-line2-with-proofing branch March 7, 2023 18:05
@jmdembe jmdembe mentioned this pull request Mar 9, 2023
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