-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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
feat(Node): add the Scheduler Node #4223
Conversation
…chedule-trigger
…chedule-trigger
…chedule-trigger
@agobrech will this replace the Cron node? |
Hi @luizeof, the scheduler is a combination of the Cron node and the Interval Node with some added features. Workflows that still have the Cron node or the Interval node will still work. You won't be able to find the old Cron node and the old Interval node in the node menu. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added 2 minor comments about it, maybe we should change those. Other than this it's all good.
We unified Cron expression generation a while ago. This change kinda reverts that by creating 2 different implementations again. |
Got released with |
* ✨ Create Schedule node with MVP structure * ✨ Add 24 increments for hours picker * 🚨 Lintfix * Add timestamp, add hour minute and cron expression * Fix bug where there was one extra interval object * Fix default value from fixedCollection * 🐛 UI fixes * 🎨 Changed logic to reflect UI fixes * Fix auto intitialising * Deprecated interval and cron in favor of schedule node * 🐛 Ui fixes * 🐛 Fix issue with week intervals * 🚨 Lint fixes * change order of days in the week to chronological order
No description provided.