Let climatisation_time_left and charging_time_left return minutes

[dvx76]

These sensors have device_class=duration so we should return an int of float with a duration unit, and not a formatted 'hh:mm' string. This will allow HA to correctly display the duration and also allow to use the sensor value in automations/calculations.

Fixes skodaconnect/homeassistant-skodaconnect#195

[dvx76]

@Farfar @WebSpider can you review this please?

[haraldpaulsen]

Can I test/review this code to speed things along?

[dvx76]

Can I test/review this code to speed things along?

Sure thing. Depending on how you're running HA you might be able to just apply the same code changes and restart HA.

[WebSpider]

Been away for work, I'll get to a review this week

[haraldpaulsen]

Sure thing. Depending on how you're running HA you might be able to just apply the same code changes and restart HA

Unfortunately I run HAOS, so I don't have access to the skodeconnect-library on disk through file editor or shell.

As @WebSpider will have a look, I can wait and have a look at doing a lab install of home assistant at a later time

[haraldpaulsen]

This is slightly offtopic, but curiosity got me..

In the Home Assistant/HACS-file /config/custom_components/skodaconnect/manifest.json
I have:
"requirements": ["skodaconnect>=1.3.5", "homeassistant>=2023.3.0"],

Could I somehow make it require the specific pull request?
ie, "git+github.com/dvx76/skodaconnect.git@fix-durations" or something like that?

Or would I end up just cluttering my python/pip-folders even if I change it back later?

[haraldpaulsen]

Been away for work, I'll get to a review this week

Did you get a chance to look at this? :-)

[WebSpider]

Thanks for the mention. I looked at it, the code looks sane, and I'll test it in my test setup against latest release today.