Skip to content
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

Fix not working on devices with Android 10 and above. #436

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

inlacou
Copy link

@inlacou inlacou commented Jan 11, 2021

In Android 10 and above, ACCESS_FINE_LOCATION is needed instead of ACCESS_COARSE_LOCATION.

@mareksip
Copy link

@inlacou @don can this be merged to master or closed?

@jriffel
Copy link

jriffel commented Mar 9, 2021

I built a test project and added https://github.com/irontec/BluetoothSerial.git directly from @mareksip as a plugin - unfortunately on my Pixel 3a Android 11 I still could not discover devices. There is never a callback as others have reported. I did check the AndroidManifest.xml and the right permissions were in there (both fine and course). I also verified the plugins source code that I had changes from @mareksip - Really bummed out we don't have bluetooth support on Cordova for Android at this point :(

@jriffel
Copy link

jriffel commented Mar 9, 2021

Well I spoke too soon... I continued testing and used adb logcat and was able to see bluetooth was being denied because location services were not enabled. Once I enabled location services I was able to scan devices using this pull request. Sorry for the misleading comment which I left in place so the internet can observe my mistakes.

@mareksip
Copy link

mareksip commented Mar 9, 2021

Well I spoke too soon... I continued testing and used adb logcat and was able to see bluetooth was being denied because location services were not enabled. Once I enabled location services I was able to scan devices using this pull request. Sorry for the misleading comment which I left in place so the internet can observe my mistakes.

Glad you found a solution! 👍

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.

3 participants