Failed upload job limiter

[ferishili]

This PR fixes #385,

What is new

• New Setting: in shared setting -> "Limit failed upload attempts"
• Notify teachers if "eventstatusnotificationenabled" is enabled.
• Puts the uploadjob into the “Archived” state when the failed upload limit is reached.
• Teachers are able to:
  • Delete archived uploadjob from the table
  • (NEW) Unarchived Icon "reload icon" is provided right beside the delete icon in the upload job list. This brings back the archived uploadjob to the initial state "Ready to upload" in order to force the cron to pick it up in the next run! aka "unarchive"

[ferishili]

There is currently a failure in upstream of moodle-plugin-ci which causes the GitHub actions to fail:

Opencast-Moodle/moodle-workflows-opencast#5

[tmtsc]

It works!

[justusdieckmann]

Thank you, this looks great to me! Apart from the notification stuff, I only have one wish/suggestion: Moodle strongly discourages the use of jQuery in new Code, so if you have the time, it would be great to use native JS instead of jQuery in new Code. But we'll have to go through the javascript code anyway someday and replace all the jQuery, so I would also merge it if you don't do that.

[ferishili]

Thank you, this looks great to me! Apart from the notification stuff, I only have one wish/suggestion: Moodle strongly discourages the use of jQuery in new Code, so if you have the time, it would be great to use native JS instead of jQuery in new Code. But we'll have to go through the javascript code anyway someday and replace all the jQuery, so I would also merge it if you don't do that.

Hi @justusdieckmann, thanks for your review.
I have added the changes and answered your question regarding redundant teacher notification.

About the JS and JQuery, I would say, because it is a part of index js file, therefore, it gets a bit messy if I change it to js now, and that would be great, if we could take care of it later on in one single go! Regarding this, I provided new es6 js for my new PR but in here, it is not feasible!

BR

[ferishili]

Hi @justusdieckmann;
thanks for your review, is there something left to do here, or we could get it merged?

[bluetom]

Hi, @ferishili,
I think it's ok for now. I will test it.