luc-ass / homebridge-evohome

Homebridge plugin for Honeywell Evohome
29 stars 16 forks source link

Fix for next schedule time (make sure time is 24h format) #25

Closed fredericvl closed 6 years ago

fredericvl commented 6 years ago

Fix for next schedule time (make sure time is 24h format)

luc-ass commented 6 years ago

no luck. same problem persists.

luc-ass commented 6 years ago

but my locale ist still AM/PM let me check wether i can change that.

luc-ass commented 6 years ago

no luck. The container is already running in a 24 hour format. It is only homebridge running in AM/PM. Any ideas on how to change that?

luc-ass commented 6 years ago
bildschirmfoto 2018-01-20 um 21 10 55

I added additional logging and found out that you are right. It seems to be the AM/PM-Problem.

luc-ass commented 6 years ago

ha! I made it work. Your idea was right, just needed to generate the right locale. The line is now set to de-DE as well as timeZone Europe/Berlin and hour12: false

fredericvl commented 6 years ago

Nice, good work!

Op 20 jan. 2018 om 22:00 heeft Lucas Gasenzer notifications@github.com het volgende geschreven:

ha! I made it work. Your idea was right, just needed to generate the right locale. The line is now set to de-DE as well as timeZone Europe/Berlin and hour12: false

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

luc-ass commented 6 years ago

Bumped version. Published on NPM (0.2.2.). Thanks again for all your hard work!