cyberjunky / home-assistant-toon_climate

This component provides a climate device for rooted Toon thermostats.
MIT License
32 stars 8 forks source link

Please add max_temp as a user configurable variable #4

Closed Ysbrand closed 4 years ago

Ysbrand commented 4 years ago

Hi Ron,

I tried to use max_temp in my configuration, but it seems that HA is not too excited about that. For now, I've just hardcoded DEFAULT_MAX_TEMP = 22.0 but it will be nice if this can be configured (there might be other properties as well (documented in https://developers.home-assistant.io/docs/entity_climate/)

cyberjunky commented 4 years ago

Yes I will add max/min temp soon, and also fix some other minor stuff.

cyberjunky commented 4 years ago

@ysbrand Can you try version 1.0.6 I added min/max temp settings.

Ysbrand commented 4 years ago

@cyberjunky : Works like a charm, thanks for making this happen!:

climate:
  - platform: toon_climate
    name: Toon Thermostaat
    host: !secret toon_ip
    port: !secret toon_port
    scan_interval: 60
    min_temp: 12
    max_temp: 22
`hvac_modes: heat, off
min_temp: 12
max_temp: 22
preset_modes: away, comfort, home, sleep
current_temperature: 19.3
temperature: 20
hvac_action: off
preset_mode: null
burner_info: 0
modulation_level: 0
friendly_name: Toon Thermostaat
supported_features: 17

Note: Before using the new settings, first restart HA with the new version of the integration.