Skip to content

LG-14277 verification data analytics event allow empty customer profile#11579

Merged
amirbey merged 2 commits intomainfrom
amirbey/LG-14277_verification_data_analytics_event
Dec 3, 2024
Merged

LG-14277 verification data analytics event allow empty customer profile#11579
amirbey merged 2 commits intomainfrom
amirbey/LG-14277_verification_data_analytics_event

Conversation

@amirbey
Copy link
Copy Markdown
Contributor

@amirbey amirbey commented Dec 3, 2024

🎫 Ticket

LG-14277

Customer profile can be blank and is currently required by analytics event which is crashing the DocVRequest job and timing out the user waiting to advance to SSN page.

🛠 Summary of changes

  • Remove requirement for customer_profile to have a value in the Socure verification data response
  • allow nil values for arguments when calling idv_socure_verification_data_requested

📜 Testing Plan

Complete DocV and advance to SSN page

@amirbey amirbey self-assigned this Dec 3, 2024
Copy link
Copy Markdown
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.

LGTM (Should have caught these on my end)

@amirbey amirbey force-pushed the amirbey/LG-14277_verification_data_analytics_event branch from bc47460 to f9bedb1 Compare December 3, 2024 15:20
@amirbey amirbey merged commit 0a42941 into main Dec 3, 2024
@amirbey amirbey deleted the amirbey/LG-14277_verification_data_analytics_event branch December 3, 2024 15:36
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