search

hoerresb / WifiWizard

A Cordova plugin for managing Wifi networks
Apache License 2.0
205 stars 177 forks source link

Call success callback after checking connection #46

Open jeffcharles opened 9 years ago

jeffcharles commented 9 years ago

I noticed when using this library that the success callback on the connect call would execute before the network was connected. These changes validate that the network is actually connected to the correct network before calling the success callback.

I'm also open to putting this new behaviour behind an optional options flag if you'd prefer to keep the old behaviour for current users.

hoerresb commented 9 years ago

Thanks for the contribution. I typically do something very similar on the JavaScript side of this to make sure I'm connected to the network. I'm going to test your code and look it over. Might be simpler to do this way.

Thanks.

jeffcharles commented 9 years ago

Would it be possible to get some sort of credit for my work in this PR? It appears it was copied and pasted into the codebase without attributing authorship.

hoerresb commented 9 years ago

Yes, sorry. Credit is due where credit is due.

I've already notice more things that need to updated. So, tonight when this gets updated I will merge your branch properly which should attribute you authorship.

tripflex commented 6 years ago

Thanks for this, I created a fork of WifiWizard (named WifiWizard2), and added this PR to it, along with credit for you in the changelog: https://github.com/tripflex/WifiWizard2