-
-
Notifications
You must be signed in to change notification settings - Fork 265
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
QR code scanning does not work on MicroG / Degoogled devices #825
Comments
It is the reason it doesn't work with MicroG. The reason I used Google's QR code scanner is that it doesn't require the camera permission. Otherwise, the ZXing library already included in app for generating QR codes can also scan them, but this is not implemented, at least for now. |
Moving to an in app solution would be nice, specially considering gms is a privacy nightmare. Not needing the camera permission is something we should at least consider giving up for the sake of better compatibility and more privacy. Forgive the previous rant, as depending on GMS for anything other than push notifications is not a good idea (and not even that at times). |
Uses a zxing-based library as a fallback when scanning a Qr-Code and GMS is not available. Closes mastodon#825
Uses a zxing-based library as a fallback when scanning a Qr-Code and GMS is not available. Closes mastodon#825
I took a look the code responsible by doing this, and it seems like we are using google's mlkit to do the work for us. Could that be the reason as for why it doesn't work on a degoogled phone?
And does that mean that we also indirectly depend on google for the qr code scanning capabilities?
The text was updated successfully, but these errors were encountered: