Add a timestamp to PeriodicSubjectFormat

cyverse-de · Oct 2, 2024 · ff72f22 · ff72f22
1 parent 35558a7
commit ff72f22
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/main.go b/main.go
@@ -216,7 +216,7 @@ func SendPeriodicNotification(ctx context.Context, j *Job) error {
 		return err
 	}
 
-	subject := PeriodicSubjectFormat // this is a static/generic string
+	subject := fmt.Sprintf(PeriodicSubjectFormat, time.Now().Format("2006-01-02 15:04")) // Mostly static with a timestamp to distinguish
 
 	msg := fmt.Sprintf(
 		PeriodicMessageFormat,

diff --git a/notifications.go b/notifications.go
@@ -44,8 +44,8 @@ const PeriodicMessageFormat = `Analysis "%s" has been running for %s and will st
 
 // PeriodicSubjectFormat is the subject for the email that is sent
 // to users as a regular reminder of a running job
-// no parameters, as it is vague on purpose to encourage opening the email with full details
-const PeriodicSubjectFormat = `CyVerse: Your analysis is still running`
+// one parameter should be a timestamp, to distinguish messages so they're not grouped by gmail et al
+const PeriodicSubjectFormat = `CyVerse: Your analysis is still running (%s)`
 
 // Notification is a message intended as a notification to some upstream service
 // or the DE UI.