-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[draytonwiser] Expose Smart Plug Power Metering, Improve null handling #9706
Conversation
Signed-off-by: James Melville <jamesmelville@gmail.com>
Signed-off-by: James Melville <jamesmelville@gmail.com>
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.
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.
LGTM
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.
LGTM
openhab#9706) * [draytonwiser] Expose Smart Plug Power Metering * [draytonwiser] Handle null values from offline devices Signed-off-by: James Melville <jamesmelville@gmail.com> Signed-off-by: John Marshall <john.marshall.au@gmail.com>
openhab#9706) * [draytonwiser] Expose Smart Plug Power Metering * [draytonwiser] Handle null values from offline devices Signed-off-by: James Melville <jamesmelville@gmail.com>
openhab#9706) * [draytonwiser] Expose Smart Plug Power Metering * [draytonwiser] Handle null values from offline devices Signed-off-by: James Melville <jamesmelville@gmail.com>
Expose Smart Plug Power Metering, Improve null handling
The Drayton Wiser API now exposes instantaneous power metering and cumulative energy usage for Wiser Smart Plug devices through the API. This PR adds channels to report this data. Capability mentioned by @andrew-schofield in this post - sorry I couldn't wait so thought I should share!
Additionally, when Wiser wireless devices are offline some properties in the API response are nulled and the binding would fail to fully update all the device's channels. Added checks to update all channels, tested with registered Smart Plug / TRV / RoomStat with power removed.