fixing handling of “immediately” setting

Vikas Agarwal · Vikas Agarwal · commit f329116b7bf2 · 2018-10-01T18:15:08.000+05:30
diff --git a/connect/notificationServices/email.js b/connect/notificationServices/email.js
@@ -288,7 +288,7 @@ function handler(topicName, messageJSON, notification) {
     }
     logger.debug('bundlePeriod=>', bundlePeriod);
 
-    if (bundlePeriod) {
+    if (bundlePeriod && "immediately" !== bundlePeriod) {
       if (!SCHEDULED_EVENT_PERIOD[bundlePeriod]) {
         throw new Error(`User's '${notification.userId}' setting for service`
           + ` '${SETTINGS_EMAIL_SERVICE_ID}' option 'bundlePeriod' has unsupported value '${bundlePeriod}'.`);

Original file line number	Diff line number	Diff line change
`@@ -288,7 +288,7 @@ function handler(topicName, messageJSON, notification) {`
`288`	`288`	`}`
`289`	`289`	`logger.debug('bundlePeriod=>', bundlePeriod);`
`290`	`290`
`291`		`- if (bundlePeriod) {`
	`291`	`+ if (bundlePeriod && "immediately" !== bundlePeriod) {`
`292`	`292`	`if (!SCHEDULED_EVENT_PERIOD[bundlePeriod]) {`
`293`	`293`	throw new Error(`User's '${notification.userId}' setting for service`
`294`	`294`	+ ` '${SETTINGS_EMAIL_SERVICE_ID}' option 'bundlePeriod' has unsupported value '${bundlePeriod}'.`);