Admin Logs - Fixed Log Settings errors (#4399)

* Reverted 4d1f352 * Showing error notif when log settings operations fail * Reverted 2b78437 * Allowing super users to add log settings * Removed check for admin role Co-authored-by: Brian Dukes <bdukes@engagesoftware.com> Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>
dnnsoftware · Jan 12, 2021 · 8543fbd · 8543fbd
1 parent b592c59
commit 8543fbd
Show file tree

Hide file tree

Showing 2 changed files with 514 additions and 516 deletions.
diff --git a/...Experience/ClientSide/AdminLogs.Web/src/components/LogSettings/LogSettingEditor/index.jsx b/...Experience/ClientSide/AdminLogs.Web/src/components/LogSettings/LogSettingEditor/index.jsx
@@ -152,7 +152,7 @@ class LogSettingEditor extends Component {
                     util.utilities.notify(Localization.get("ConfigUpdated"));
                     props.Collapse(event);
                 }, () => {
-                    util.utilities.notify(Localization.get("ConfigUpdateError"));
+                    util.utilities.notifyError(Localization.get("ConfigUpdateError"));
                 }));
             } else {
                 props.dispatch(LogSettingActions.addLogSetting(logSettingDetail, () => {
@@ -175,15 +175,15 @@ class LogSettingEditor extends Component {
                     util.utilities.notify(Localization.get("ConfigDeleted"));
                     props.Collapse(event);
                 }, () => {
-                    util.utilities.notify(Localization.get("DeleteError"));
+                    util.utilities.notifyError(Localization.get("DeleteError"));
                 })
                 );
             }, () => {
                 util.utilities.notify(Localization.get("ConfigDeleteCancelled"));
             });
         }
         else {
-            util.utilities.notify(Localization.get("ConfigDeleteInconsistency"));
+            util.utilities.notifyError(Localization.get("ConfigDeleteInconsistency"));
         }
     }