Bug Fix: Crasher when trying to use the "Notify Me" button inside running jobs. Fixes #156.

jhuckaby · jhuckaby · commit 51edb9260f8a · 2026-02-22T09:18:07.000-08:00
diff --git a/lib/api/jobs.js b/lib/api/jobs.js
@@ -522,7 +522,7 @@ class Jobs {
 			
 			if (!notify_me) {
 				// add notification
-				job.actions.push({ condition: 'complete', type: 'email', email: user.email, enabled: true, hidden: true });
+				job.actions.push({ condition: 'complete', type: 'email', email: user.email, users: [], enabled: true, hidden: true });
 				notify_me = true;
 			}
 			else {

Original file line number	Diff line number	Diff line change
`@@ -522,7 +522,7 @@ class Jobs {`
`522`	`522`
`523`	`523`	`if (!notify_me) {`
`524`	`524`	`// add notification`
`525`		`- job.actions.push({ condition: 'complete', type: 'email', email: user.email, enabled: true, hidden: true });`
	`525`	`+ job.actions.push({ condition: 'complete', type: 'email', email: user.email, users: [], enabled: true, hidden: true });`
`526`	`526`	`notify_me = true;`
`527`	`527`	`}`
`528`	`528`	`else {`