Add notification for successful archive action

ManageIQ · Jun 5, 2018 · cb5962b · cb5962b
1 parent 59fded6
commit cb5962b
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 8 deletions.
diff --git a/app/javascript/react/screens/App/Overview/Overview.js b/app/javascript/react/screens/App/Overview/Overview.js
@@ -242,7 +242,8 @@ class Overview extends React.Component {
       allArchivedPlanRequestsWithTasks,
       archiveTransformationPlanAction,
       archiveTransformationPlanUrl,
-      isFetchingArchivedTransformationPlans
+      isFetchingArchivedTransformationPlans,
+      addNotificationAction
     } = this.props;
 
     const inProgressRequestsTransformationMappings = () => {
@@ -347,6 +348,7 @@ class Overview extends React.Component {
               }
               archiveTransformationPlanAction={archiveTransformationPlanAction}
               archiveTransformationPlanUrl={archiveTransformationPlanUrl}
+              addNotificationAction={addNotificationAction}
             />
           )}
           {hasSufficientProviders ? (

diff --git a/app/javascript/react/screens/App/Overview/components/Migrations/Migrations.js b/app/javascript/react/screens/App/Overview/components/Migrations/Migrations.js
@@ -29,7 +29,8 @@ const Migrations = ({
   archivedTransformationPlans,
   allArchivedPlanRequestsWithTasks,
   archiveTransformationPlanAction,
-  archiveTransformationPlanUrl
+  archiveTransformationPlanUrl,
+  addNotificationAction
 }) => {
   const filterOptions = [
     MIGRATIONS_FILTERS.notStarted,
@@ -131,6 +132,7 @@ const Migrations = ({
               archiveTransformationPlanUrl={archiveTransformationPlanUrl}
               fetchTransformationPlansAction={fetchTransformationPlansAction}
               fetchTransformationPlansUrl={fetchTransformationPlansUrl}
+              addNotificationAction={addNotificationAction}
             />
           )}
           {activeFilter === MIGRATIONS_FILTERS.archived && (
@@ -170,7 +172,8 @@ Migrations.propTypes = {
   allArchivedPlanRequestsWithTasks: PropTypes.array,
   isFetchingArchivedTransformationPlans: PropTypes.string,
   archiveTransformationPlanAction: PropTypes.func,
-  archiveTransformationPlanUrl: PropTypes.string
+  archiveTransformationPlanUrl: PropTypes.string,
+  addNotificationAction: PropTypes.func
 };
 Migrations.defaultProps = {
   transformationPlans: [],

diff --git a/app/javascript/react/screens/App/Overview/components/Migrations/MigrationsCompletedList.js b/app/javascript/react/screens/App/Overview/components/Migrations/MigrationsCompletedList.js
@@ -17,6 +17,7 @@ const MigrationsCompletedList = ({
   archiveTransformationPlanUrl,
   fetchTransformationPlansAction,
   fetchTransformationPlansUrl,
+  addNotificationAction,
   archived
 }) => (
   <Grid.Col xs={12}>
@@ -93,12 +94,18 @@ const MigrationsCompletedList = ({
                 archiveTransformationPlanUrl,
                 plan.id
               )
-                .then(() =>
-                  fetchTransformationPlansAction({
+                .then(() => {
+                  addNotificationAction({
+                    message: sprintf(__('%s successfully archived'), plan.name),
+                    notificationType: 'success',
+                    persistent: false
+                  });
+
+                  return fetchTransformationPlansAction({
                     url: fetchTransformationPlansUrl,
                     archived: false
-                  })
-                )
+                  });
+                })
                 .then(() => {
                   hideConfirmModalAction();
                 });
@@ -213,7 +220,8 @@ MigrationsCompletedList.propTypes = {
   archiveTransformationPlanAction: PropTypes.func,
   archiveTransformationPlanUrl: PropTypes.string,
   fetchTransformationPlansAction: PropTypes.func,
-  fetchTransformationPlansUrl: PropTypes.string
+  fetchTransformationPlansUrl: PropTypes.string,
+  addNotificationAction: PropTypes.func
 };
 MigrationsCompletedList.defaultProps = {
   finishedTransformationPlans: [],