Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Chef][RPC] Missing support for BooleanState sensor testing #31725

Closed
tcarmelveilleux opened this issue Jan 28, 2024 · 0 comments · Fixed by #31745
Closed

[Chef][RPC] Missing support for BooleanState sensor testing #31725

tcarmelveilleux opened this issue Jan 28, 2024 · 0 comments · Fixed by #31745
Labels
examples testing

Comments

@tcarmelveilleux
Copy link
Contributor

  • Google (and others) need testing support of BooleanState events for such things contact sensors.
  • Pigweed RPC don't have a way to generate changes of BooleanState sensors
tcarmelveilleux added a commit to tcarmelveilleux/connectedhomeip that referenced this issue Jan 28, 2024
@tcarmelveilleux
[WIP] Adding boolean state sensor control to Chef
393662c 
- Add new RPCs to set/read BooleanState cluster
- Update nRF, ESP32 and Linux chef examples to include
- Update some docs that were stale
- Allow StatusUtils for RPC to deal with CHIP_ERROR

Issue project-chip#25225
Fixes project-chip#31725
@tcarmelveilleux tcarmelveilleux mentioned this issue Jan 29, 2024
Merged
@mergify mergify bot closed this as completed in #31745 Jan 29, 2024
mergify bot pushed a commit that referenced this issue Jan 29, 2024
@tcarmelveilleux @erwinpan1 @restyled-commits
Add BooleanState cluster control to pigweed RPCs (#31745)
adcf742 
* [WIP] Adding boolean state sensor control to Chef

- Add new RPCs to set/read BooleanState cluster
- Update nRF, ESP32 and Linux chef examples to include
- Update some docs that were stale
- Allow StatusUtils for RPC to deal with CHIP_ERROR

Issue #25225
Fixes #31725

* Support BooleanState in RPC console

* Restyled by clang-format

* Fix ESP32 build

* Rename EmberOrChipStatusToPwStatus to ToPwmStatus

---------

Co-authored-by: erwinpan1 <erwinpan1@gmail.com>
Co-authored-by: Restyled.io <commits@restyled.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
examples testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add BooleanState cluster control to pigweed RPCs tcarmelveilleux/connectedhomeip
1 participant
@tcarmelveilleux