Skip to content
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

Can't setup notification in some device #93

Closed
Zkffkah opened this issue Nov 10, 2016 · 3 comments
Closed

Can't setup notification in some device #93

Zkffkah opened this issue Nov 10, 2016 · 3 comments
Assignees
Labels
bug Bug that is caused by the library

Comments

@Zkffkah
Copy link

Zkffkah commented Nov 10, 2016

Summary

For some device, we can't not setup ble notification using current implementation of ble notification setup in the library.

However, if I change
bluetoothGatt.writeDescriptor(bluetoothGattDescriptor);
to
bluetoothGatt.setWriteType(BluetoothGattCharacteristic.WRITE_TYPE_DEFAULT);
bluetoothGatt.writeDescriptor(bluetoothGattDescriptor);

the notification is set up correctly.

I think it's a known bug as also stated and solved in the below repo.
Android-nRF-Toolbox

@dariuszseweryn dariuszseweryn added the bug Bug that is caused by the library label Nov 10, 2016
@dariuszseweryn dariuszseweryn self-assigned this Nov 10, 2016
dariuszseweryn added a commit that referenced this issue Nov 10, 2016
… type. #93

Reviewers: michal.zielinski, pawel.urban

Reviewed By: pawel.urban

Differential Revision: https://phabricator.polidea.com/D1946
@dariuszseweryn
Copy link
Owner

Will be fixed in the next build of SNAPSHOT release (around 10 minutes or so).
Thanks for the info!

@Zkffkah
Copy link
Author

Zkffkah commented Nov 11, 2016

Great. But it seems the snapshot build failed for some checkstyle problems (travis), please check with that. Thanks!

@dariuszseweryn
Copy link
Owner

I find the checkstyle slightly annoying... I will add it to my local build script.
Now it works. Thanks for the info!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug that is caused by the library
Projects
None yet
Development

No branches or pull requests

2 participants