-
Notifications
You must be signed in to change notification settings - Fork 523
CHAD-17017: Add the ReadAttrbitue or refresh code for the initial state #2656
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
base: main
Are you sure you want to change the base?
Conversation
|
Invitation URL: |
Test Results 71 files 480 suites 0s ⏱️ Results for commit 05cce9c. ♻️ This comment has been updated with latest results. |
|
zigbee-illuminance-sensor_coverage.xml
zigbee-switch_coverage.xml
zigbee-window-treatment_coverage.xml
Minimum allowed coverage is Generated by 🐒 cobertura-action against 05cce9c |
|
@inasail looks like you have a test failing |
61ec8df to
e7ab0e6
Compare
|
@greens Thank you. I fixed it. Sorry for the mistake. |
|
@greens Please merge this PR. Thank you~ |
| device:send(LockCluster.attributes.LockState:read(device)) | ||
| device:send(PowerConfiguration.attributes.BatteryPercentageRemaining:read(device)) |
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.
These should already be read during the added lifecycle event, which calls refresh.
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.
Oh, you are right. I will remove those codes.
b001022 to
05cce9c
Compare
https://smartthings.atlassian.net/browse/CHAD-17017
Add refresh/readAttribute logic to retrieve the initial state fast.