cgarwood / homeassistant-zwave_mqtt

Limited Pre-Release of the new OZW1.6 Z-Wave component. Currently has limited platform support. Check the README for more details.
72 stars 8 forks source link

Small fixes #87

Closed marcelveldt closed 4 years ago

marcelveldt commented 4 years ago

A bunch of small fixes so we can close some of the reported issue. Most important fix is that we're now able to survive a restart of the OZW daemon.

fixes #25 fixes #82 fixes #85

MartinHjelmare commented 4 years ago

Is this PR related to https://github.com/cgarwood/homeassistant-zwave_mqtt/issues/24?

MartinHjelmare commented 4 years ago

I think we would benefit of splitting this PR into multiple PRs, one per task.

marcelveldt commented 4 years ago

Is this PR related to #24?

Correct, and a few other very small requests/improvements I fixed while at it.

marcelveldt commented 4 years ago

I think we would benefit of splitting this PR into multiple PRs, one per task.

Yeah, maybe better after all. I'll create new PR's cherry picking from this one. The commits regarding the entity removal will be redone. If you can give me a quick pointer how to delete an entity the right way that would be perfect.