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

Change “Heating to” to “Heat to” #2

Open
Bleh55 opened this issue Dec 11, 2021 · 4 comments
Open

Change “Heating to” to “Heat to” #2

Bleh55 opened this issue Dec 11, 2021 · 4 comments
Labels
enhancement New feature or request

Comments

@Bleh55
Copy link

Bleh55 commented Dec 11, 2021

The icon in Home.app says “Heating to xxx” and always throws me off thinking it is actively heating. Does it make sense to change that to “Heat to xxx”? That also matches the way thermostats display when it is not actively heating.

@Bleh55 Bleh55 added the enhancement New feature or request label Dec 11, 2021
@dustindclark
Copy link
Owner

Yeah, unfortunately, this is HomeKit's thermostat behavior, and there's nothing that can be done either in this plugin or in Homebridge.

@dustindclark
Copy link
Owner

Reopening..looks like the Nest plugin handles this better. Will see if I can follow that pattern.

@dustindclark dustindclark reopened this Dec 12, 2021
@tedinorbit
Copy link

If you want it to look "idle" (thus say "Heat to..." instead of "Heating to...", and will be a green icon instead of orange), this

bindStaticValues() {
        this.service.updateCharacteristic(this.platform.Characteristic.TemperatureDisplayUnits, this.platform.getConfig().temperatureUnits === constants_1.TemperatureUnits.F
            ? this.platform.Characteristic.TemperatureDisplayUnits.FAHRENHEIT
            : this.platform.Characteristic.TemperatureDisplayUnits.CELSIUS);
        this.service.updateCharacteristic(this.platform.Characteristic.CurrentHeatingCoolingState, this.platform.Characteristic.CurrentHeatingCoolingState.HEAT);
    }

can be changed to this:

bindStaticValues() {
        this.service.updateCharacteristic(this.platform.Characteristic.TemperatureDisplayUnits, this.platform.getConfig().temperatureUnits === constants_1.TemperatureUnits.F
            ? this.platform.Characteristic.TemperatureDisplayUnits.FAHRENHEIT
            : this.platform.Characteristic.TemperatureDisplayUnits.CELSIUS);
        this.service.updateCharacteristic(this.platform.Characteristic.CurrentHeatingCoolingState, this.platform.Characteristic.CurrentHeatingCoolingState.OFF);
    }

bindStaticValues() {
this.service.updateCharacteristic(this.platform.Characteristic.TemperatureDisplayUnits,
this.platform.getConfig().temperatureUnits === TemperatureUnits.F
? this.platform.Characteristic.TemperatureDisplayUnits.FAHRENHEIT
: this.platform.Characteristic.TemperatureDisplayUnits.CELSIUS);
this.service.updateCharacteristic(this.platform.Characteristic.CurrentHeatingCoolingState,
this.platform.Characteristic.CurrentHeatingCoolingState.HEAT);
}

@zimuliu
Copy link

zimuliu commented May 16, 2024

implemented in #26

oyarzun pushed a commit to oyarzun/homebridge-rinnai-controlr that referenced this issue Aug 7, 2024
Use proper rounding for controllers with imperial units
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants