Merge pull request #2426 from skateman/error-danger-notifications

Notification type error should be rendered in patternfly as danger
ManageIQ · Oct 17, 2017 · e0b0998 · e0b0998
2 parents 83ed40f + 57c2527
commit e0b0998
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/app/assets/javascripts/services/event_notifications_service.js b/app/assets/javascripts/services/event_notifications_service.js
@@ -47,6 +47,13 @@ function eventNotifications($timeout, API) {
     });
   };
 
+  var levelToType = function(level) {
+    if (level === 'error') {
+      return 'danger';
+    }
+    return level;
+  };
+
   this.doReset = function(seed) {
     state.groups.splice(0, state.groups.length);
     var events = {
@@ -117,7 +124,7 @@ function eventNotifications($timeout, API) {
       id: id,
       notificationType: notificationType,
       unread: true,
-      type: type,
+      type: levelToType(type),
       message: message,
       data: notificationData,
       href: id ? '/api/notifications/' + id : undefined,
@@ -154,7 +161,7 @@ function eventNotifications($timeout, API) {
         if (showToast) {
           notification.unread = true;
         }
-        notification.type = type;
+        notification.type = levelToType(type);
         notification.message = message;
         notification.data = notificationData;
         notification.timeStamp = (new Date()).getTime();