bastelgarage / dual-led-ring-watch

3D Printed Wall-Mounted NeoPixel LED Clock with NTP Synchronization
MIT License
54 stars 4 forks source link

fix timezone bug #4

Open Flet opened 10 months ago

Flet commented 10 months ago

Hi!

We ran into a bug similar to what was described in #3 by @ctobio

This update uses the hour and minute from the local time variable instead of timeClient directly.

It also looks like the timeClient is instantiated with an offset of 3600 seconds (+1 hour), so timeClient.getHour() was returning an adjusted time, which probably made it difficult to see that the local time variable was not being used at all!

Hope this helps others. Thanks for putting this project together!