search

evothings / cordova-ble

Bluetooth Low Energy plugin for Cordova
http://www.evothings.com/
Apache License 2.0
242 stars 103 forks source link

Support indicate notifications on Android #102

Closed ghost closed 8 years ago

ghost commented 8 years ago

Reportedly works on iOS but not on Android (most likely because notify notifications are turned on automatically when enabling notifications, Android plugin code needs to be updated to also support indicate notifications).

ghost commented 8 years ago

Added support for indications in this commit: https://github.com/evothings/cordova-ble/commit/2349d5f285c5454a10df740bbfe4b15d5b5bc6d2

Let me know what you think.

ghost commented 8 years ago

Here is a version of Evothings Viewer for Android that supports indications:

https://evothings.com/uploads/evothings-studio/viewer/EvothingsViewer_1.4.1_debug_160705_1637.apk

ghost commented 8 years ago

Updated version of Evothings Viewer for Android that supports indications:

https://evothings.com/uploads/evothings-studio/viewer/EvothingsViewer_1.4.1_debug_160705_2056.apk

Updated BLE plugin version 1.4.0:

https://github.com/evothings/cordova-ble/releases/tag/1.4.0

Updated EasyBLE library:

https://github.com/evothings/evothings-libraries/blob/master/libs/evothings/easyble/easyble.js

ghost commented 8 years ago

What remains before closing this issue is further testing.

ghost commented 8 years ago

I just discovered a bug that prevented automatic writing of the config descriptor from working when enabling notifications/indications.

The options parameter should also work now.

Published a new BLE plugin version 1.4.1 with a fix.

ghost commented 8 years ago

Release candidate of Evothings Viewer 1.4.1 for Android for testing available here:

https://evothings.com/uploads/evothings-studio/viewer/EvothingsViewer_1.4.1_160711_1608.apk

Contains the BLE plugin 1.4.1 with the bug fix for notifications.

ghost commented 8 years ago

Closing this issue, let me know if to reopen, if any problems should arise.