search

kookob / plugin

Intellij Plugin Document.
https://plugins.jetbrains.com/organization/abc
Other
3 stars 3 forks source link

CRON generator works inappropriate #14

Closed IVoyt closed 7 months ago

IVoyt commented 1 year ago

The ending number stands for execution day and 1 means Monday while plugin renders Sunday

image
kookob commented 1 year ago

The ending number 1-7 means SUN-SAT, The first day of the week is Sunday, not Monday. and 1 is used to represent Sunday here.

IVoyt commented 1 year ago

Well that is confusing. Cron valid values for day of week are 0-6 (Sun-Sat). Where did you take 1-7 is correct?

Вт, 2 мая 2023 г. в 14:44, ob @.***>:

The ending number 1-7 means SUN-SAT, The first day of the week is Sunday, not Monday. and 1 is used to represent Sunday here.

— Reply to this email directly, view it on GitHub https://github.com/kookob/plugin/issues/14#issuecomment-1531326568, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACTLNTENHMF6IJOEXLJJNX3XEDXRLANCNFSM6AAAAAAXS5MKDM . You are receiving this because you authored the thread.Message ID: @.***>

kookob commented 1 year ago

Sorry, There may be some error here. For linux and Spring, the allowable values are 0-7, where 0 and 7 represent Sunday. For Quartz, the allowable values are 1-7, where 1 represents Sunday. My Toolset plugin use Quartz implement. So incompatible with all here. A better approach is to directly specify the day of the week. 0 0 1 ? * Mon https://stackoverflow.com/questions/18919151/crontab-day-of-the-week-syntax https://en.wikipedia.org/wiki/Cron

IVoyt commented 1 year ago

Got it, thank you for the explanation

вт, 2 мая 2023 г. в 15:25, ob @.***>:

Sorry, There may be some error here. For linux and Spring, the allowable values are 0-7, where 0 and 7 represent Sunday. For Quartz, the allowable values are 1-7, where 1 represents Sunday. My Toolset plugin use Quartz implement. So incompatible with all here. A better approach is to directly specify the day of the week. 0 0 1 ? * Mon https://stackoverflow.com/questions/18919151/crontab-day-of-the-week-syntax https://en.wikipedia.org/wiki/Cron

— Reply to this email directly, view it on GitHub https://github.com/kookob/plugin/issues/14#issuecomment-1531389907, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACTLNTGLD35Y4J5AH3WM7XDXED4M7ANCNFSM6AAAAAAXS5MKDM . You are receiving this because you authored the thread.Message ID: @.***>

kookob commented 7 months ago

I apologize for the delay in responding to this issue. I want to inform you that the new version of the toolset plugin now supports the Unix cron expression. Thank you.

IVoyt commented 7 months ago

Hi. Thank you, I've noticed that in one of the recent changelogs :)

пн, 4 дек. 2023 г. в 13:41, ob @.***>:

I apologize for the delay in responding to this issue. I want to inform you that the new version of the toolset plugin now supports the Unix cron expression. Thank you.

— Reply to this email directly, view it on GitHub https://github.com/kookob/plugin/issues/14#issuecomment-1838461409, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACTLNTFE6OXBPI2N74BW3WLYHWZHTAVCNFSM6AAAAAAXS5MKDOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZYGQ3DCNBQHE . You are receiving this because you authored the thread.Message ID: @.***>