-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Revert Change that Didnt Fix Blackscreen #10332
Conversation
…calls so user will have more relevant hint text
@@ -513,15 +513,11 @@ function AcuantCapture( | |||
|
|||
function onSelfieCaptureOpen() { | |||
trackEvent('idv_sdk_selfie_image_capture_opened', { captureAttempts }); | |||
|
|||
setImageCaptureText(''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not actually removing this, it's just moving into the acuant-selfie-camera onOpened
function
setIsCapturingEnvironment(true); | ||
} | ||
|
||
function onSelfieCaptureClosed() { | ||
trackEvent('idv_sdk_selfie_image_capture_closed_without_photo', { captureAttempts }); | ||
|
||
setImageCaptureText(''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not actually removing this, it's just moving into the acuant-selfie-camera onClosed
function
@@ -110,6 +112,7 @@ function AcuantSelfieCamera({ | |||
}, | |||
onPhotoTaken: () => { | |||
// The photo has been taken and it's showing a preview with a button to accept or retake the image. | |||
onImageCaptureFeedback(''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is getting added back, removed in the earlier PR.
Not sure what does following meant? @charleyf
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what does following meant? @charleyf
It also adds an announcement (via hint text) when the selfie is captured.
@dawei-nava This is a leftover sentence that no longer reflects what I'm doing in the PR. Removed! Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Tested selfie function on mobile, got hint texts, though not updated as fast as idea, but I think that's with the SDK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and saw the same as Dawei. Hint text is there! It is slow but I think that is the case in main right now.
🛠 Summary of changes
This PR rolls back the changes here because they didn't fix the blackscreen issue.