Skip to content

Add electrical measurement sensor to ZHA#14561

Merged
fabaff merged 8 commits intohome-assistant:devfrom
dmulcahey:dm/electrical-sensor
May 26, 2018
Merged

Add electrical measurement sensor to ZHA#14561
fabaff merged 8 commits intohome-assistant:devfrom
dmulcahey:dm/electrical-sensor

Conversation

@dmulcahey
Copy link
Copy Markdown
Contributor

Description:

Add sensor that polls electricity usage from devices such as the Samsung smart outlet

Comment thread homeassistant/components/sensor/zha.py Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comparison to None should be 'if cond is None:'

@fabaff fabaff changed the title zha: add electrical measurement sensor Add electrical measurement sensor to ZHA May 20, 2018
@rcloran rcloran self-requested a review May 20, 2018 23:15
Copy link
Copy Markdown
Contributor

@rcloran rcloran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. #14563 will conflict with this, so waiting for checks there to complete and will land and then fix up here.

@dshokouhi
Copy link
Copy Markdown
Member

Just wanted to comment saying the code changes are working for me, thank you so much :)

@syssi
Copy link
Copy Markdown
Member

syssi commented May 22, 2018

Please debase this PR or fix the merge conflict. Thanks in advance!

@dmulcahey dmulcahey requested review from a team, Danielhiversen, cdce8p and syssi as code owners May 26, 2018 12:13
@dmulcahey dmulcahey requested a review from a team May 26, 2018 12:13
@dmulcahey dmulcahey force-pushed the dm/electrical-sensor branch from 96ad0df to 938cc81 Compare May 26, 2018 13:13
@fabaff fabaff merged commit eae9726 into home-assistant:dev May 26, 2018
@balloob balloob mentioned this pull request Jun 8, 2018
girlpunk pushed a commit to girlpunk/home-assistant that referenced this pull request Sep 4, 2018
* Add electrical measurement sensor

* correct state update

* hound fix

* zha: Add metering sensor (home-assistant#14562)

* Add IlluminanceMeasurementSensor to ZHA (home-assistant#14563)

* add IlluminanceMeasurementSensor

* address review comment

* Fix whitespace error during merge

* Add electrical measurement sensor

* correct state update

* hound / flake8
@home-assistant home-assistant locked and limited conversation to collaborators Sep 5, 2018
@dmulcahey dmulcahey deleted the dm/electrical-sensor branch December 26, 2018 12:12
@ghost ghost removed the platform: sensor.zha label Mar 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants