Fix: .confirmOnly requires reset() call, but .testAndConfirm requires success() call at the end of Confirm Callback

NordicSemiconductor / IOS-nRF-Connect-Device-Manager

A mobile management library for devices supporting nRF Connect Device Manager.

https://www.nordicsemi.com/Software-and-tools/Software/nRF-Connect-SDK

Apache License 2.0

97 stars 41 forks source link

Fix: .confirmOnly requires reset() call, but .testAndConfirm requires success() call at the end of Confirm Callback #28

Closed dinesharjani closed 2 years ago

dinesharjani commented 2 years ago

This was all prompted by a Pull Request by a user. And yes, we did do the check to see that with reset(), after successful reconnection a success() call is sent up through the API ranks.