-
Notifications
You must be signed in to change notification settings - Fork 41
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
Allow Transak to use more device sensors (fixes E2E tests) #1832
Conversation
Deployed to Cloudflare Pages
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1832 +/- ##
==========================================
- Coverage 81.19% 81.12% -0.08%
==========================================
Files 192 192
Lines 5053 5053
Branches 933 933
==========================================
- Hits 4103 4099 -4
- Misses 950 954 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
|
allow="camera;microphone;fullscreen;payment" | ||
// Expands on https://github.com/Transak/transak-sdk/blob/2ebb3bd/src/index.js#L52 | ||
// and somewhat matches https://docs.transak.com/docs/web-integration#embediframe-webapp | ||
allow="accelerometer;camera;microphone;fullscreen;gyroscope;payment" |
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 weird why they need motion detection.
If we are changing feature policy to make tests happy should we do it only for process?.env?.NODE_ENV === 'test'
?
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.
It could be for video liveness check (does device movement match the video), but the event listeners get attached much earlier by https://api.sardine.ai/assets/loader.min.js
I'm leaning towards increasing permissions, since we don't have insight into whether this decreases users' ability to complete KYC
No description provided.