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.

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

Bugfix. climate and covermqt #3130

Merged
merged 1 commit into from
Sep 2, 2016
Merged

Conversation

turbokongen
Copy link
Contributor

@turbokongen turbokongen commented Sep 2, 2016

Description:

  1. Move fetching unit of measurement back to sensor. It never worked properly fetching from setpoint.
  2. mqtt wrong value was wathced for the state. state never updated.

Related issue (if applicable): fixes #
#3090
#3052

@turbokongen turbokongen force-pushed the zwave_fixes branch 3 times, most recently from 92c4e78 to b181878 Compare September 2, 2016 05:24
@robbiet480 robbiet480 added this to the 0.27.2 milestone Sep 2, 2016
@turbokongen turbokongen force-pushed the zwave_fixes branch 2 times, most recently from 0279d6e to 4dec7f5 Compare September 2, 2016 08:46
@Teagan42
Copy link
Contributor

Teagan42 commented Sep 2, 2016

LGTM

@robbiet480
Copy link
Member

🐬

@robbiet480 robbiet480 merged commit a7a662d into home-assistant:dev Sep 2, 2016
robbiet480 added a commit that referenced this pull request Sep 2, 2016
@robbiet480 robbiet480 mentioned this pull request Sep 2, 2016
robbiet480 added a commit that referenced this pull request Sep 3, 2016
* Host should be optional for apcupsd component (#3072)

* Zwave climate Bugfix: if some setpoints have different units, we should fetch the o… (#3078)

* Bugfix: if some setpoints have different units, we should fetch the one that are active.

* Move order of population for first time detection

* Default to config if None unit_of_measurement

* unit fix (#3083)

* humidity slider (#3088)

* If device was off target temp was null. Default to Heating setpoint (#3091)

* Fix for BLE device tracker (#3019)

* Bug fix tracked devices
* Added scan_duration configuration parameter

* fix homematic climate implementation (#3114)

* Allow 'None' MAC to be loaded from known_devices (#3102)

* Climate and cover bugfix (#3097)

* Avoid None comparison for zwave cover.

* Just rely on unit from config for unit_of_measurement

* Explicit return None

* Mqtt (#11)

* Explicit return None

* Missing service and wrong service name defined

* Mqtt state was inverted, and never triggering

* Fixed Homematic cover (#3116)

* Add missing docstrings (fix PEP257 issues) (#3098)

* Add missing docstrings (fix PEP257 issues)

* Finish sentence

* Merge pull request #3130 from turbokongen/zwave_fixes

Bugfix. climate and covermqt

* Back out insteon hub and fan changes (#3062)

* Bump version

* Special frontend build for 0.27.2
@home-assistant home-assistant locked and limited conversation to collaborators Mar 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants