-
Notifications
You must be signed in to change notification settings - Fork 103
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
Disable camera for not HTTPS #306 #307
Conversation
Note: the localization strings added only for English and Russian languages. |
@@ -74,6 +74,9 @@ uploadcare.namespace 'locale.translations', (ns) -> | |||
notFound: | |||
title: 'No camera detected' | |||
text: 'Looks like you have no camera connected to this device.' | |||
notHttps: | |||
title: 'Camera does not allowed for not protected connections' |
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.
to be or not to be
There are should be two separate pull-requests for Camera and intialization errors. |
Ok, I`ll fix it tonight |
The main Idea was to disable camera source at all. I.e. to hide entire tab, not to show error message to the user. Sorry for not make it clear. Error message is meaningless for end-user, because he can't do anything with this. Site developer should see the message and decide to exclude the camera tab or to switch to the HTTPS. |
Maybe almost all code should be in |
Done with task.
Added check for chrome and for http connection according task
Spent 3 hours