BlueAndi / Pixelix

Full RGB LED matrix, based on an ESP32 and WS2812B LEDs.
MIT License
300 stars 59 forks source link

[Feature] Configurable "start of week" for DateTimePlugin lamps #177

Closed ryannevell closed 1 month ago

ryannevell commented 2 months ago

The DateTimePlugin currently assumes that Monday is the first day of the week:

    /* tm_wday starts at sunday, first lamp indicates monday.*/
    uint8_t activeLamp = (0U < timeInfo.tm_wday) ? (timeInfo.tm_wday - 1U) : (DateTimePlugin::MAX_LAMPS - 1U);

In many locales, this is not the case, so having this be a configurable option for this plugin would be nice.

BlueAndi commented 1 month ago

I am currently on a business trip, I try to check it after I am back. Thx.