Skip to content
This repository has been archived by the owner on Feb 4, 2024. It is now read-only.

Fix callbacks for channel 0 #159

Merged
merged 1 commit into from
Aug 26, 2018

Conversation

klada
Copy link
Contributor

@klada klada commented Aug 26, 2018

Events can be fired for entities and channels, so we also need to
register callbacks for the HMChannel instance - even on channel 0.
This fixes the updating of the UNREACH flag, which until now is
only updated when reloading pyhomematic.

See danielperna84/pyhomematic#121

Events can be fired for entities and channels, so we also need to
register callbacks for the HMChannel instance - even on channel 0.
This fixes the updating of the `UNREACH` flag, which until now is
only updated when reloading pyhomematic.

See danielperna84/pyhomematic#121
@danielperna84
Copy link
Owner

Thanks. It's been a while since I worked on this topic, so I'm unsure if this has possible sideffects. I'll just assume you have tested this thoroughly and everything keeps working for you as it used to. 😉

@danielperna84 danielperna84 merged commit 39170b9 into danielperna84:master Aug 26, 2018
@danielperna84
Copy link
Owner

Could you please recreate the PR? You had it pointing to the master branch, not the dev branch, and I merged it without checking. Hence I reverted the merge to keep the master branch in sync with the last release.

@klada
Copy link
Contributor Author

klada commented Aug 26, 2018

@danielperna84 Sorry about the wrong branch. I assumed that the master branch is for "mainline" development and "dev" is your personal branch. Since you have created a PR template now this is clear :) I have just created a new PR (see #161).

Yes, I have tested this in my environment (CCU2) and it did not have any side effects. The UNREACH event can now be handled properly by hass. The homematic component needed some additional treatment for this (home-assistant/core#16202), but it was very minor and not directly related to this PR.

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.

2 participants