-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Correct thread status callback functionality #9971
Conversation
@KariHaapalehto, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How is this fixing it ? I can see removing connection status and status changed ? Can you expand your commit msg
At the start-up, there was 2 NSAPI_STATUS_CONNECTING callbacks, so extra one removed from ThreadInterface.cpp. |
At the start-up, there was 2 NSAPI_STATUS_CONNECTING callbacks, so extra one removed from ThreadInterface.cpp. At the network lost case, there was NSAPI_STATUS_DISCONNECTED and NSAPI_STATUS_LOCAL_UP callbacks. NSAPI_STATUS_DISCONNECTED has been removed, since the NSAPI_STATUS_LOCAL_UP is enought.
e6390e3
to
d944836
Compare
@KariHaapalehto what is the status of this PR ? |
@adbridge This one was waiting on us. Commit message was updated a while ago. |
CI started |
Test run: FAILEDSummary: 1 of 13 test jobs failed Failed test jobs:
|
restarted exporter |
Correct thread status callback functionality
Description
Status callback correct.
Tested locally with k64f using thread mesh network.
Pull request type
Reviewers
Release Notes