You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The issue is that #30004 added the BC feature from the spec, but did not actually set that feature bit in the places that need it set. Which is probably all the .zap files that correspond to the .matter files listed by git grep -l "handle command OpenBasicCommissioningWindow"
Did that PR actually add that command? It doesn't appear as changed in the .matter files, but there's also a LOT of them and I could just be missing it.
I recall going through these a while ago and removing BC command and feature specifically. The fix that was applied then was to remove the command rather than set the feature bit. BC is mentioned in the thread model, so the sense was that we shouldn't have it in the examples as there's a lot of copy-paste into real products where this could be a problem.
Feature Area
Other
Test Case
TC-IDM-10.2
Reproduction steps
The issue is that #30004 added the BC feature from the spec, but did not actually set that feature bit in the places that need it set. Which is probably all the .zap files that correspond to the .matter files listed by
git grep -l "handle command OpenBasicCommissioningWindow"
Bug prevalence
Always
GitHub hash of the SDK that was being used
47097e0
Platform
core
Anything else?
No response
The text was updated successfully, but these errors were encountered: