Skip to content

LG-11427 add phone with camera to image upload#9537

Merged
amirbey merged 8 commits intomainfrom
amirbey/LG-11427-add-phone-with-camera-to-image-upload
Nov 7, 2023
Merged

LG-11427 add phone with camera to image upload#9537
amirbey merged 8 commits intomainfrom
amirbey/LG-11427-add-phone-with-camera-to-image-upload

Conversation

@amirbey
Copy link
Contributor

@amirbey amirbey commented Nov 3, 2023

🎫 Ticket

LG-11427

🛠 Summary of changes

  1. upload form to send phone_with_camera in payload
  2. image upload form to log phone_with_camera value in analytics

📜 Testing Plan

Complete document capture in the IdV and verify the following events contain a phone_with_camera attribute:
IdV: doc auth image upload form submitted
IdV: doc auth image upload vendor submitted

@amirbey amirbey changed the title Amirbey/lg 11427 add phone with camera to image upload LG-11427 add phone with camera to image upload Nov 3, 2023
@amirbey amirbey force-pushed the amirbey/LG-11427-add-phone-with-camera-to-image-upload branch from f0b1dda to 2618d46 Compare November 3, 2023 22:23
@amirbey amirbey force-pushed the amirbey/LG-11427-add-phone-with-camera-to-image-upload branch from c81e28e to 9118f54 Compare November 6, 2023 19:52
@amirbey amirbey marked this pull request as ready for review November 6, 2023 20:40
@amirbey amirbey self-assigned this Nov 6, 2023
@amirbey amirbey requested review from a team, charleyf, dawei-nava and night-jellyfish and removed request for a team November 6, 2023 21:23
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.

This looks like what we talked about today. 👍

@amirbey amirbey merged commit 1db5d83 into main Nov 7, 2023
@amirbey amirbey deleted the amirbey/LG-11427-add-phone-with-camera-to-image-upload branch November 7, 2023 01:26
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