patch

Author: lbr38

www/public/resources/js/task.js

@@ -55,6 +55,8 @@ $(document).on('change','input:radio[name="task-schedule-type"]',function () {
         $('.task-schedule-form-params[action="' + action + '"]').find('.task-schedule-recurring-day-input').hide();
         $('.task-schedule-form-params[action="' + action + '"]').find('.task-schedule-recurring-monthly-input').hide();
         $('.task-schedule-form-params[action="' + action + '"]').find('.task-schedule-recurring-cron-input').hide();
+        $('input[type="checkbox"][param-name="schedule-notify-error"]').prop('checked', true);
+        $('input[type="checkbox"][param-name="schedule-notify-success"]').prop('checked', true);
     }
 
     /**
@@ -64,6 +66,9 @@ $(document).on('change','input:radio[name="task-schedule-type"]',function () {
         $('.task-schedule-form-params[action="' + action + '"]').find('.task-schedule-recurring-frequency-input').show();
         $('.task-schedule-form-params[action="' + action + '"]').find('.task-schedule-unique-input').hide();
         $('.task-schedule-form-params[action="' + action + '"]').find('.task-schedule-recurring-cron-input').hide();
+        // Disable notification on success by default for recurring tasks to avoid spamming users with notifications
+        $('input[type="checkbox"][param-name="schedule-notify-error"]').prop('checked', true);
+        $('input[type="checkbox"][param-name="schedule-notify-success"]').prop('checked', false);
     }
 }).trigger('change');
 

www/views/includes/forms/tasks/schedule.inc.php

@@ -91,8 +91,8 @@
 
             <div class="task-schedule-recurring-cron-input hide">
                 <h6 class="required">CRON EXPRESSION</h6>
+                <p class="note">Format: minute hour day month weekday.</p>
                 <input type="text" class="task-param" param-name="schedule-cron" placeholder="*/15 * * * *" />
-                <p class="note">Format: minute hour day month weekday</p>
             </div>
             <?php
         endif;