Skip to content
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: Added Common Humidifer/Dehumidifer Settings to Thermostat and Fans #1699

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nberardi
Copy link
Contributor

I am using Scrypted to pull my thermostat settings into Home Assistant via the HomeKit Controller and noticed that they require the following characteristics to be set to show the Fan Mode and Humidity settings on their climate card.

  • Characteristic.TargetFanState as seen here to enable Fan Modes.
  • Characteristic.CurrentFanState as seen here to indicate if the fan is active
  • Characteristic.TargetRelativeHumidity as seen here to enable Humidity card.
  • Characteristic.CurrentRelativeHumidity as seen here to indicate humidity to card.

I also took the opportunity to move the humidity settings to a common setup method for both Thermostat and Fan/

return service;
}

export function addDehumidifer(device: ScryptedDevice & HumiditySetting & HumiditySensor, accessory: Accessory): Service {
Copy link
Owner

Choose a reason for hiding this comment

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

I don't think this needs export

return service;
}

export function addHumidifier(device: ScryptedDevice & HumiditySetting & HumiditySensor, accessory: Accessory): Service {
Copy link
Owner

Choose a reason for hiding this comment

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

I don't think this needs export

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants