-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[homekit] Allow configuring thermostat modes via metadata #17056
Conversation
for example, if it's a water heater that's always in heat mode, so you don't have to create dummy items for the characteristics this also adds the infrastructure necessary to use metadata characteristics as mandatary characteristics it also adds TemperatureUnit as a characteristic the user can provide, and support TargetRelativeHumidity characteristic for Thermostat Signed-off-by: Cody Cutrer <cody@cutrer.us>
This pull request has been mentioned on openHAB Community. There might be relevant details there: https://community.openhab.org/t/homekit-integration-beta/157125/1 |
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.
good feature.
pretty code heavy change for one feature, but you also did some code re-org to make cleaner. thank you.
LGTM
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.
Thanks, some minor comments.
...mekit/src/main/java/org/openhab/io/homekit/internal/accessories/HomekitAccessoryFactory.java
Outdated
Show resolved
Hide resolved
...mekit/src/main/java/org/openhab/io/homekit/internal/accessories/HomekitAccessoryFactory.java
Outdated
Show resolved
Hide resolved
...n/java/org/openhab/io/homekit/internal/accessories/HomekitMetadataCharacteristicFactory.java
Outdated
Show resolved
Hide resolved
Co-authored-by: lsiepel <leosiepel@gmail.com> Signed-off-by: Cody Cutrer <cody@cutrer.us>
for example, if it's a water heater that's always in heat mode, so you don't have to create dummy items for the characteristics
this also adds the infrastructure necessary to use metadata characteristics as mandatary characteristics
it also adds TemperatureUnit as a characteristic the user can provide, and support TargetRelativeHumidity characteristic for Thermostat