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
Ignore some HomeKit devices #15316
Ignore some HomeKit devices #15316
Conversation
There are some devices that speak HomeKit that we shouldn't expose. Some bridges (such as the Hue) provide reduced functionality over HomeKit and have a functional native API, so should be ignored. We also shouldn't offer to configure the built-in Home Assistant HomeKit bridge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Is this a breaking change? We should probably document this at least in our docs. |
It would break anyone who was using a Tradfri via Homekit (Hue doesn't currently work due to a bug in the protocol library). I /doubt/ that anyone's actually doing that, but it seems reasonable to document. |
Please highlight the breaking change describing paragraph so we can copy that to the release notes. |
There are some devices that speak HomeKit that we shouldn't expose. Some bridges (such as the Hue) provide reduced functionality over HomeKit and have a functional native API, so should be ignored. We also shouldn't offer to configure the built-in Home Assistant HomeKit bridge.
There are some devices that speak HomeKit that we shouldn't expose. Some bridges (such as the Hue) provide reduced functionality over HomeKit and have a functional native API, so should be ignored. We also shouldn't offer to configure the built-in Home Assistant HomeKit bridge.
Wouldn't adding the Hue bridge via HomeKit be more responsive than the current local polling integration? |
There are some devices that speak HomeKit that we shouldn't expose. Some
bridges (such as the Hue) provide reduced functionality over HomeKit and
have a functional native API, so should be ignored. We also shouldn't
offer to configure the built-in Home Assistant HomeKit bridge.
Breaking change: Tradfri will no longer be automatically detected by the HomeKit controller. Use the native integration instead.
Checklist:
tox
. Your PR cannot be merged unless tests pass