Skip to content

Don't dispatch to components when there are no channels for ZHA sensors#21223

Merged
pvizeli merged 2 commits into
home-assistant:devfrom
dmulcahey:dm/zha-dont-dispatch-if-no-listeners
Feb 20, 2019
Merged

Don't dispatch to components when there are no channels for ZHA sensors#21223
pvizeli merged 2 commits into
home-assistant:devfrom
dmulcahey:dm/zha-dont-dispatch-if-no-listeners

Conversation

@dmulcahey
Copy link
Copy Markdown
Contributor

fixes last issue in #21136

@ghost ghost assigned dmulcahey Feb 19, 2019
@ghost ghost added the in progress label Feb 19, 2019
@dmulcahey dmulcahey force-pushed the dm/zha-dont-dispatch-if-no-listeners branch from a0c1e16 to dd05ed3 Compare February 19, 2019 21:56
@dmulcahey dmulcahey force-pushed the dm/zha-dont-dispatch-if-no-listeners branch from dd05ed3 to 387242c Compare February 19, 2019 21:56
Copy link
Copy Markdown
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Is this a fix for 0.88?

Comment thread homeassistant/components/zha/core/gateway.py Outdated
@dmulcahey
Copy link
Copy Markdown
Contributor Author

@MartinHjelmare same as the other PR. #21196 would have to be pulled in to .88 for this to be included. I think it makes sense to pull in all 3...

@dmulcahey dmulcahey added this to the 0.88.0 milestone Feb 20, 2019
@pvizeli pvizeli merged commit 5b24b27 into home-assistant:dev Feb 20, 2019
@ghost ghost removed the in progress label Feb 20, 2019
@pvizeli pvizeli removed this from the 0.88.0 milestone Feb 20, 2019
@dmulcahey
Copy link
Copy Markdown
Contributor Author

@pvizeli why was this removed from the .88 milestone?

@balloob balloob added this to the 0.88.0 milestone Feb 20, 2019
@balloob
Copy link
Copy Markdown
Member

balloob commented Feb 20, 2019

Adding it back to the milestone - this was discussed between @dmulcahey and me (and then not properly communicated)

balloob pushed a commit that referenced this pull request Feb 20, 2019
…rs (#21223)

* don't dispatch when channels don't exist

* review comment
thibmaek added a commit to thibmaek/home-assistant that referenced this pull request Feb 21, 2019
* upstream/dev: (948 commits)
  Prevent partial custom component overlays (home-assistant#21070)
  Alarm trigger support for Point (home-assistant#21207)
  Upgrade opensensemap-api to 0.1.4 (home-assistant#21240)
  Remove constraint from regex (home-assistant#21239)
  Updated frontend to 20190220.0
  Support OpenWRT 18.06 in luci device tracker (home-assistant#21236)
  Bump zigpy (home-assistant#21203)
  Fix bug in ZHA and tweak non sensor channel logic (home-assistant#21234)
  Don't dispatch to components when there are no channels for ZHA sensors (home-assistant#21223)
  Add zone and reps for Xiaomi vacuum (home-assistant#19777)
  Fix an Ambient PWS exception when location info is missing (home-assistant#21220)
  Add self to integration sensor and utility_meter (home-assistant#21226)
  Prevent invalid context from crashing (home-assistant#21231)
  Update pyhomematic to 0.1.56 (home-assistant#21227)
  Fix the build (home-assistant#21229)
  Updated frontend to 20190219.0
  Add yeelight flow action support (home-assistant#21195)
  Refactor ZHA listeners into channels (home-assistant#21196)
  Fix Homematic IP Cloud configuration (home-assistant#21202)
  Push pyads to 3.0.7 (home-assistant#21216)
  ...
@dmulcahey dmulcahey deleted the dm/zha-dont-dispatch-if-no-listeners branch May 8, 2019 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants