ARMmbed / mbed-os-example-ble

BLE demos using mbed OS and mbed cli
Apache License 2.0
134 stars 118 forks source link

BLE_GapButton example doesn't work #261

Closed pan- closed 3 years ago

pan- commented 5 years ago

I've noticed several issues on the example:

The advertising payload isn't updated into its advertising set; the following lines are missing in the function update_button_payload

https://github.com/ARMmbed/mbed-os-example-ble/blob/47a653482689be385512aa522570c343b8471843/BLE_GAPButton/source/main.cpp#L116-L119

Event with this change update doesn't occur on DISCO_L475VG_IOT01A . However it works on NRF52840_DK.

On NRF52840 the button doesn't work out of the box.

Finally, the field _button_count is not initialised properly.

@desmond-blue @donatieng This is blocking @ashok-rao .

ciarmcom commented 4 years ago

Thank you for raising this issue. Please note we have updated our policies and now only defects should be raised directly in GitHub. Going forward questions and enhancements will be considered in our forums, https://forums.mbed.com/ . If this issue is still relevant please re-raise it there. This GitHub issue will now be closed.

adbridge commented 4 years ago

Apologies, this was closed in error by the automated bot - re-opening.

ciarmcom commented 3 years ago

Thank you for raising this detailed GitHub issue. I am now notifying our internal issue triagers. Internal Jira reference: https://jira.arm.com/browse/IOTOSM-2062

pan- commented 3 years ago

Similar to #215

paul-szczepanek-arm commented 3 years ago

This example is being replaced. The new example will not have this issue.