Fixed error response when updating group

TIER · Oct 25, 2021 · cc9cd4b · cc9cd4b
1 parent 006b7fb
commit cc9cd4b
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 2 deletions.
diff --git a/ui/src/app/admin/container/MetadataActions.js b/ui/src/app/admin/container/MetadataActions.js
@@ -2,7 +2,7 @@ import React from 'react';
 import { DeleteConfirmation } from '../../core/components/DeleteConfirmation';
 import { useMetadataActivator, useMetadataEntity } from '../../metadata/hooks/api';
 
-import { NotificationContext, createNotificationAction } from '../../notifications/hoc/Notifications';
+import { NotificationContext, createNotificationAction, NotificationTypes } from '../../notifications/hoc/Notifications';
 
 export function MetadataActions ({type, children}) {
 
@@ -21,6 +21,12 @@ export function MetadataActions ({type, children}) {
                 `Metadata ${type} has been ${enabled ? 'enabled' : 'disabled'}.`
             ));
             cb();
+        } else {
+            const { errorCode, errorMessage, cause } = activator?.response?.data;
+            dispatch(createNotificationAction(
+                `${errorCode}: ${errorMessage} ${cause ? `-${cause}` : ''}`,
+                NotificationTypes.ERROR
+            ));
         }
     }
 
@@ -31,6 +37,12 @@ export function MetadataActions ({type, children}) {
                 `Metadata ${type} has been deleted.`
             ));
             cb();
+        } else {
+            const { errorCode, errorMessage, cause } = activator?.response?.data;
+            dispatch(createNotificationAction(
+                `${errorCode}: ${errorMessage} ${cause ? `-${cause}` : ''}`,
+                NotificationTypes.ERROR
+            ));
         }
     }
 

diff --git a/ui/src/app/dashboard/view/SourcesTab.js b/ui/src/app/dashboard/view/SourcesTab.js
@@ -6,7 +6,7 @@ import SourceList from '../../metadata/domain/source/component/SourceList';
 import { useMetadataEntities, useMetadataEntity } from '../../metadata/hooks/api';
 import { Search } from '../component/Search';
 
-import { NotificationContext, createNotificationAction } from '../../notifications/hoc/Notifications';
+import { NotificationContext, createNotificationAction, NotificationTypes } from '../../notifications/hoc/Notifications';
 
 const searchProps = ['serviceProviderName', 'entityId', 'createdBy'];
 
@@ -42,6 +42,12 @@ export function SourcesTab () {
         if (updater.response.ok) {
             dispatch(createNotificationAction(`Updated group successfully.`));
             loadSources();
+        } else {
+            const { errorCode, errorMessage, cause } = updater?.response?.data;
+            dispatch(createNotificationAction(
+                `${errorCode}: ${errorMessage} ${cause ? `-${cause}` : ''}`,
+                NotificationTypes.ERROR
+            ));
         }
     }