-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Android] Use GetDeviceBeingCommissioned for network commissioning #11658
Conversation
PR #11658: Size comparison from 9964cdb to c0d4a94 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
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.
Great ~ Looks good to me.
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.
Fast tracking, given platform changes by a platform maintainer.
@austinh0 Note: Still needs zap regenerate |
/rebase |
c0d4a94
to
d713f31
Compare
PR #11658: Size comparison from dbe2a62 to d713f31 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
…roject-chip#11658) * Use GetDeviceBeingCommissioned for network commissioning * Restyled by clang-format Co-authored-by: Restyled.io <[email protected]>
Problem
GetConnectedDevice
in EnterNetworkFragment looks for the device operationally, but it's not on the network.Change overview
GetDeviceBeingCommissioned
instead. This is synchronous so removed the coroutine/suspend bits from EnterNetworkFragment.Testing