Merge pull request #2608 from skateman/notification-data-type

Display the missing notification type icons in the drawer (cherry picked from commit 151e84f) https://bugzilla.redhat.com/show_bug.cgi?id=1509414
ManageIQ · Nov 20, 2017 · cae6bb2 · cae6bb2
1 parent 3654e9b
commit cae6bb2
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/app/assets/javascripts/controllers/notifications/notifications_drawer_controller.js b/app/assets/javascripts/controllers/notifications/notifications_drawer_controller.js
@@ -99,14 +99,14 @@ function NotificationsDrawerCtrl($scope, eventNotifications, $timeout) {
 
   vm.customScope.getNotficationStatusIconClass = function(notification) {
     var retClass = '';
-    if (notification && notification.data && notification.data.type) {
-      if (notification.data.type == 'info') {
+    if (notification && notification.type) {
+      if (notification.type == 'info') {
         retClass = "pficon pficon-info";
-      } else if ((notification.data.type == 'error') || (notification.data.type == 'danger')) {
+      } else if ((notification.type == 'error') || (notification.type == 'danger')) {
         retClass = "pficon pficon-error-circle-o";
-      } else if (notification.data.type == 'warning') {
+      } else if (notification.type == 'warning') {
         retClass = "pficon pficon-warning-triangle-o";
-      } else if ((notification.data.type == 'success') || (notification.data.type == 'ok')) {
+      } else if ((notification.type == 'success') || (notification.type == 'ok')) {
         retClass = "pficon pficon-ok";
       }
     }