-
-
Notifications
You must be signed in to change notification settings - Fork 243
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[mainui] CronBuilder puts monday as day 1 #902
Comments
There are multiple side-effects to this "deviation from the rest of the world":
|
We could change the core cron scheduler (I actually did some testing on that). But that means it breaks all existing cronjobs. So this would need to be discussed. Wdyt? |
Note that for 1. above I checked and it actually already puts the unambiguous abbreviations when selecting "Specific day of week". [Btw I seem to remember that the scheduler doesn't support every option here - esp. the last 5 ones, some work but I'm pretty sure "nearest weekday" (1W-5W) doesn't.]
It would break all existing affected cronjobs, not sure if there are that many.
I think we could afford to introduce one. |
In the main ui the cronbuilder puts day 1 as Monday. However, openHAB-core has Sunday as day 1. This differences probably is caused by the fact that openHAB used the Quartz library in the past, while the rest of the world seems to put Monday as day 1.
See also this discussion: https://community.openhab.org/t/oh3-cron-trigger-1-day-out/ So unless we decide to change this to use Monday as day 1 this is a UI bug.
The text was updated successfully, but these errors were encountered: