Code review comment for lp:~edb/quam-plures/cronjob_stuff

Revision history for this message
Tilman Blumenbach (tblue) wrote :

Marking as Needs Fixing, see above:

> Looks fine, but why do you set $ctst_error to 0 (used to be 1) when setting
> the default filters for the cronjob list? This way, failed cronjobs are hidden
> by default; shouldn't the admin see them right away?

review: Needs Fixing

« Back to merge proposal