bugy / script-server

Web UI for your scripts with execution management
Other
1.52k stars 244 forks source link

Schedule repeat end date / execution count #676

Closed UrekD closed 12 months ago

UrekD commented 1 year ago

Hey,

For scheduling repeatable task it would be nice to have extra options to specify when the task should stop/not be executed anymore or how many times a task should execute.

For I instance I reworked the current repeatable option to require the end time to satisfy our needs as: image

I'm open to working on a proper implementation, but I'd need someone to do the frontend, as I'm terrible with that.

bugy commented 1 year ago

Hi @UrekD great idea, thanks. Feel free to submit PR for backend. You can try to do frontend too, it should be pretty simple in this case. I could tell you which classes are affected

UrekD commented 1 year ago

Hi @UrekD great idea, thanks. Feel free to submit PR for backend. You can try to do frontend too, it should be pretty simple in this case. I could tell you which classes are affected

Hey @bugy, yeah I figured it would be much easier to edit the vue and build it than the manual js editing I did like an idiot before xD

After managing to get it to build on windows, I managed to construct the box shown below.

brave_dpBB3Fl6x2

If you are happy with the structure of that I'll move onto the functional part. As you can see the styling is not good with that, if there is an easy way to handle that I'd appreciate that. The yet unformatted additions can be found here.

bugy commented 1 year ago

Looks very good to me! If you struggle too much with styling, you can leave it for me

UrekD commented 1 year ago

Minor bug the date field is not required, so user can sent it empty as it just grabs the default value.

image

UrekD commented 1 year ago

Looks very good to me! If you struggle too much with styling, you can leave it for me

Ah I thought the styling was dynamic not static, in this case I got it.

brave_XAUBlsHBCw

I think I got the functionality down I'll give it a day or 2 test then open merge 😄