-
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] Add Fabric ID parameter #22094
[Android] Add Fabric ID parameter #22094
Conversation
PR #22094: Size comparison from 48f87f3 to 7284c79 Increases (6 builds for cc13x2_26x2, nrfconnect, psoc6, telink)
Decreases (4 builds for bl602, cc13x2_26x2, psoc6)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
1.0 justification: platform-specific change (android) providing significant value (missing functionality added) |
PR #22094: Size comparison from fa24ab6 to 344d16a Increases (4 builds for bl602, cc13x2_26x2, cyw30739, psoc6)
Decreases (6 builds for cc13x2_26x2, cyw30739, psoc6, telink)
Full report (32 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
@joonhaengHeo this needs style fixes. |
… support_android_multiple_fabricid
…engHeo/connectedhomeip into support_android_multiple_fabricid
I ran restyle script. Thanks a lot. |
PR #22094: Size comparison from b4776ad to 2dfe132 Increases (3 builds for psoc6)
Decreases (3 builds for cyw30739, telink)
Full report (32 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
* Draft: fixes related to PASE-only commissioning on Android * fix build * avoid null pointer in cleanup
* Add fabricId parameter in andorid controller * restyle locally
* Add fabricId parameter in andorid controller * restyle locally Co-authored-by: joonhaengHeo <[email protected]>
* Add fabricId parameter in andorid controller * restyle locally
Problem
What is being fixed?
Change overview
Add Controller Parameter : Fabric ID - default value is 1.
Testing
How was this tested? (at least one bullet point required)