Skip to content

Remove discovered MQTT climate device when discovery topic is cleared#16856

Merged
balloob merged 1 commit intohome-assistant:devfrom
emontnemery:mqtt_discovery_remove_climate
Sep 25, 2018
Merged

Remove discovered MQTT climate device when discovery topic is cleared#16856
balloob merged 1 commit intohome-assistant:devfrom
emontnemery:mqtt_discovery_remove_climate

Conversation

@emontnemery
Copy link
Copy Markdown
Contributor

Description:

Support removal of previously discovered MQTT climate device when a (retained) discovery payload is cleared.

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox.

Comment thread tests/components/climate/test_mqtt.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.

trailing whitespace

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

continuation line missing indentation or outdented

@emontnemery emontnemery force-pushed the mqtt_discovery_remove_climate branch from bf127c9 to 00600f5 Compare September 25, 2018 14:40
@balloob balloob merged commit 0dbfd77 into home-assistant:dev Sep 25, 2018
@ghost ghost removed the in progress label Sep 25, 2018
@emontnemery emontnemery deleted the mqtt_discovery_remove_climate branch September 27, 2018 17:37
@balloob balloob mentioned this pull request Oct 12, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Feb 5, 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.

4 participants