CJNE / ha-myenergi

Home Assistant integration for MyEnergi devices
MIT License
150 stars 33 forks source link

No adjustment for DST #347

Open rz210 opened 1 year ago

rz210 commented 1 year ago

myenergi data through the integration appears with no adjustment for daylight savings time. With Home Assistant showing correctly elsewhere data for myenergi is showing with a -1 offset for my timezone.

G6EJD commented 1 year ago

That’s because MyEnergi store all data in UTC

rz210 commented 1 year ago

I get that, but should the integration not account for DST?

CJNE commented 1 year ago

Hmm, could you elaborate a little more on this, I don't think any dates are being handled in the process. What entity do you get this issue with?

rz210 commented 1 year ago

Hmm, could you elaborate a little more on this, I don't think any dates are being handled in the process. What entity do you get this issue with?

The issue only appeared to affect the ME data which led me to believe that it was the integration. However, I found that it showed correctly on the homeassistant app. Ultimately it appeared to be an issue with Firefox that was resolved after an update. I saw mentions on the homeassistant forums of issues seemingly with other integrations also. Apologies.

G6EJD commented 1 year ago

So isn’t this issue now closed?