-
-
Notifications
You must be signed in to change notification settings - Fork 324
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
Upgrade documentation for the cron and timer hives #293
Comments
👍 The intention of both hives hasn't been clear to me either. I've been thinking we had two different implementations that did the same thing for a while. |
@muesli, would it help to rename that hive to "Once", "On This Date" or something like that? |
I heard you are reworking the admin interface (potentially adding categories) but I think that it is quite important that since this two hives are so close (utility, logo, input, descriptions) that they need to share part of the name and be differentiated on an other part. That way they will be next to each other in the list and as a user I will ask myself which one I need right now. Except if you intent to randomize the hives order… Something like: |
I agree, we should clean that up and combine those two Hives into one, which then offers multiple event types. We should probably also drop the underlying cron library and switch to a maintained and better tested one: https://github.com/robfig/cron |
Handles parsing better, supporting the cron spec. Partially addresses #293
I just saw there are Cron and Time hives and they look too be identical.
The descriptions should better state that Cron is recurring and Time one time only ? The Time hive may need a "once" or "one time only" in the description to make the difference really clear.
And if possible they should be near (Time once, Time recurring (cron)) so user will see both at once and choose whichever they need.
The text was updated successfully, but these errors were encountered: