muesli / beehive

A flexible event/agent & automation system with lots of bees 🐝
GNU Affero General Public License v3.0
6.3k stars 324 forks source link

Upgrade documentation for the cron and timer hives #293

Open Zykino opened 4 years ago

Zykino commented 4 years ago

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.

rubiojr commented 4 years ago

:+1: 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.

rubiojr commented 4 years ago

@muesli, would it help to rename that hive to "Once", "On This Date" or something like that?

Zykino commented 4 years ago

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: Time (Once) Time (Recurrent)

muesli commented 4 years ago

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