-
Notifications
You must be signed in to change notification settings - Fork 145
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
Add support for API level 29 #104
Add support for API level 29 #104
Conversation
Looks good to me! |
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!
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.
Remove all the random spaces
d803f8f
to
ac1efbf
Compare
f0fa119
to
f01ee4c
Compare
I have amended space removal fix into the existing commit to keep the history clean |
f01ee4c
to
e617350
Compare
Please merge |
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.
Looks good to me, I haven't been able to test myself but if others report this works I think it should be good to go
I tested on my Devices. Looks fine. |
From this conversation I assume that PR is ready to merge and is waiting almost month to do it. Could you review and merge the changes as it is quite important feature to support latest android version? |
I'm very new to android development, so take this with a pinch of salt. I've noticed that in many places API 29 will keep using This is causing many places to malfunction. For example mapping of SSID to network id doesn't work at all. I can give you more details, in case you confirm that this is indeed a problem and not just me badly misunderstanding the code. |
is there any news with this PR ? in November 2020 will no longer have the option to use this plugin without this PR. |
Does this fix work for anyone? I tried connecting to my network (in this case a wifi enabled device that is not on the internet) and a box pops up like it is trying to ask me if I want to connect to it but nothing is in the list. Is there anyway for this to happen in the background, or have the user give it permission to connect without asking the next time? |
@markarupert unfortunately no way to get rid of the box, please take a look at this https://blog.ostebaronen.dk/2019/11/android-10-wifi.html If you are looking at connecting to an iOT device with no internet i have a forked version of ViggoChavine's branch that adds that support, please look at that. |
@VijayVaveHealth what's the changes you made that are specific for internetless IOT? |
@saoron this is what i have in my network request builder
|
Pending reviewer @arsenal942, please review & approve :-) |
Looks like @arsenal942 first approved, but then reset the vote because some changes were requested. @tripflex are you able to remove @arsenal942 as a reviewer, ping him to approve, or force approve this PR? |
After that my app worked as expected with android sdk 29+. |
Sorry guys, so little time to work on this stuff, merging! |
works fine for me |
Description of the Change
Java: adds support for API level 29 released with Android 10. Uses WifiNetworkSpecifier.Builder to establish a connection, and ConnectivityManager.unregisterNetworkCallback when disconnecting
Javascript: does not add double apostrophe when Android platform 10 or above because the new method for connecting does not support it
Benefits
Able to use the plugin on Android devices having API level 29 or higher
Possible Drawbacks
According to our tests none. Changes are only applied for API version 29 and higher