NordicSemiconductor / pc-ble-driver-py

Python bindings for the ble-driver library
Other
126 stars 115 forks source link

Feature/ble gatts char md fix #188

Closed dangu closed 3 years ago

dangu commented 3 years ago

This is a branch aiming to fix the issue discussed in this thread: https://devzone.nordicsemi.com/f/nordic-q-a/70813/user-description-in-pc-ble-driver-py

The current status is that the user description can be written and the number of bytes are correct but the actual bytes are garbage.

CLAassistant commented 3 years ago

CLA assistant check
All committers have signed the CLA.

bihanssen commented 3 years ago

@dangu this PR overlaps with #184, should 184 be closed?

dangu commented 3 years ago

@bihanssen I would like to keep them separated as #184 and #188 are handling different issues. #188 needs more work as the actual bytes written show up as garbage. Should I better rebase #188 to get rid of the overlapping commits?

bihanssen commented 3 years ago

@dangu ok no problem, let's keep them separate then.