search

ponewheel / android-ponewheel

pOneWheel Android app
MIT License
71 stars 25 forks source link

Quick ble fix #54

Closed kwatkins closed 6 years ago

kwatkins commented 6 years ago

@ebabel ui notifications was broxe on master, was getting: W/BluetoothGatt: Unhandled exception in callback java.lang.IllegalArgumentException: Invalid UUID string: MockOnewheelCharacteristicSpeed at java.util.UUID.fromString(UUID.java:194) at net.kwatts.powtools.util.BluetoothUtilImpl$1.onServicesDiscovered(BluetoothUtilImpl.java:121) at android.bluetooth.BluetoothGatt$1$5.run(BluetoothGatt.java:312) at android.bluetooth.BluetoothGatt.runOrQueueCallback(BluetoothGatt.java:725) at android.bluetooth.BluetoothGatt.-wrap0(Unknown Source:0) at android.bluetooth.BluetoothGatt$1.onSearchComplete(BluetoothGatt.java:307) at android.bluetooth.IBluetoothGattCallback$Stub.onTransact(IBluetoothGattCallback.java:110) at android.os.Binder.execTransact(Binder.java:697)

Adding in a check for the isnotify flags skipped this but you should review in case this breaks the test/mocks your working on.

ebabel commented 6 years ago

@kwatkins the Dialog is just a white box for me. Because I was in night mode.

ebabel commented 6 years ago

I moved this branch to be on master without the donation branch. Closing this PR and will move comments to the appropriate PR.