Add internal notifications to notification drawer

- previously only a curated list of API Events made it into the drawer - this change catches all "internal events" currently experienced as toasts & brings them into the drawer - future improvement may be to suppress the "toast" for certain internal events so as not to excessively message the user
openshift · Sep 19, 2017 · 966b580 · 966b580
1 parent 2041c75
commit 966b580
Show file tree

Hide file tree

Showing 6 changed files with 302 additions and 286 deletions.
diff --git a/app/scripts/directives/notifications/notificationCounter.js b/app/scripts/directives/notifications/notificationCounter.js
@@ -32,7 +32,7 @@
         if(!projectName) {
           return;
         }
-        notificationListeners.push($rootScope.$on('NotificationDrawerWrapper.count', cb));
+        notificationListeners.push($rootScope.$on('NotificationDrawerWrapper.onUnreadNotifications', cb));
       };
 
       var deregisterNotificationListeners = function() {