Fix requireInteraction by setting priority to 0 instead of 2. Also added a context menu option to test notification popups. (#22)

Author: richadams

pd-notifier/background/pd-notifier.js

@@ -94,6 +94,8 @@ function PagerDutyNotifier()
     // This will handle the event triggered from clicking one of the notification's buttons.
     self.handlerButtonClicked = function handlerButtonClicked(notificationId, buttonIndex)
     {
+        if (notificationId == 'test') { return; } // Ignore for test notifications.
+
         switch (buttonIndex)
         {
             case 0: // Acknowledge
@@ -117,6 +119,7 @@ function PagerDutyNotifier()
     // This will handle the event triggered when clicking on the main notification area.
     self.handlerNotificationClicked = function handlerNotificationClicked(notificationId)
     {
+        if (notificationId == 'test') { return; } // Ignore for test notifications.
         window.open('https://' + self.account + '.pagerduty.com/incidents/' + notificationId);
     }
 
@@ -252,7 +255,7 @@ function PagerDutyNotifier()
             title: incident.summary,
             message: "Service: " + incident.service.summary,
             contextMessage: incident.urgency.charAt(0).toUpperCase() + incident.urgency.slice(1) + " Urgency",
-            priority: 2,
+            priority: 0,
             isClickable: true,
             buttons: buttons,
             requireInteraction: self.requireInteraction
@@ -335,6 +338,29 @@ chrome.contextMenus.onClicked.addListener(function(info, tab)
     }
 });
 
+// Add option to trigger a test notification popup.
+chrome.contextMenus.create({
+    title: "Show test notification",
+    id: "pd_test_notification",
+    contexts: ["browser_action"],
+    visible: true
+});
+
+chrome.contextMenus.onClicked.addListener(function(info, tab)
+{
+    if (info.menuItemId === "pd_test_notification")
+    {
+      _pdNotifier.triggerNotification({
+        'id': 'test',
+        'summary': 'Test Notification',
+        'service': {
+          'summary': 'Test Service'
+        },
+        'urgency': 'high'
+      })
+    }
+});
+
 // Listen for Chrome Alarms and retrigger the notifier when one is caught.
 chrome.alarms.onAlarm.addListener(function(alarm)
 {

pd-notifier/manifest.json

@@ -3,7 +3,7 @@
     "name": "PagerDuty Notifier",
     "short_name": "PD Notifier",
     "description": "Desktop notifications for your PagerDuty incidents.",
-    "version": "0.22",
+    "version": "0.23",
     "author": "Rich Adams (https://richadams.me)",
     "icons": {
          "16": "images/icon-16.png",