search

jmcollin78 / versatile_thermostat

A full featured Thermostat for Home Assistant: presets, window, motion, presence and overpowering management
MIT License
328 stars 34 forks source link

[5.0.1] Preset not saved #284

Closed MaxWaldorf closed 11 months ago

MaxWaldorf commented 11 months ago

Once configured a first time, the values of the preset configurations are not shown or saved (?).

Once configured, if I go back to configuration of the device, the values show all 0...

image

jmcollin78 commented 11 months ago

Hello @MaxWaldorf,

I am very surprise of this. Have you some error logs ? Can you send me the exact sequence of screen reproducting the issue please ?

MaxWaldorf commented 11 months ago

Here are all windows:

image image image image image image image image image image image image image

jmcollin78 commented 11 months ago

The last screen that says "Succeed " is missing. All configuration change are saved at this step. If you stop before the last screen, nothing is saved.

Is that your point ?

MaxWaldorf commented 11 months ago

No, it saves as expected but my point is that if I change the configuration then all values show up execpt preset that is 0 everywhere...

I would have thought it would bring back the former values I set...

jmcollin78 commented 11 months ago

Yes it should bring back the previous values of the presets.

jmcollin78 commented 11 months ago

I just try and you are right the values are resetted to 0 when I come back on the same screen. Thank you for the report !

jmcollin78 commented 11 months ago

The presets are well saved, they are "just" not recalled on the update screen.

jmcollin78 commented 11 months ago

Done in main branch (ora wait for next release)

jmcollin78 commented 11 months ago

https://github.com/jmcollin78/versatile_thermostat/releases/tag/5.1.0