$cordovaBLE improvements

[tomasbedrich]

• added startScan and stopScan methods to be able to scan until something is discovered
• removed promises from startNotification method - it doesn't make sense there
• fixed some other methods to be up to date with latest cordova-plugin-ble-central plugin
• updated mocks

[tomasbedrich]

Should solve #981

[keytuff]

i like to write like this:

startNotification: function (deviceID, serviceUUID, characteristicUUID) {
var q = $q.defer();
ble.startNotification(deviceID, serviceUUID, characteristicUUID, function (result) {
q.notify(result);
}, function (error) {
q.reject(error);
});
return q.promise;
}

so you can use it in notify function of pormise like $CordovaBLE.Scan().

[tomasbedrich]

This way you would create a promise, which is never resolved. I think it is
not the ideal solution. If you have a specific use case, you can wrap the
call to startNotification to your code returning a promise.

On Tue, Jan 19, 2016, 02:31 keyrratuff [email protected] wrote:

i like to write like this:

startNotification: function (deviceID, serviceUUID, characteristicUUID) {
var q = $q.defer();
ble.startNotification(deviceID, serviceUUID, characteristicUUID, function
(result) {
q.notify(result);
}, function (error) {
q.reject(error);
});
return q.promise;
}

so you can use it in notify function of pormise like $CordovaBLE.Scan().

—
Reply to this email directly or view it on GitHub
#1079 (comment).