logic: slight update to when the index closed callout is shown

We only show the index closed callout in the flyout when the reindex operation is not considered "completed"
elastic · Mar 6, 2020 · a0e0ad4 · a0e0ad4
1 parent 83e18d8
commit a0e0ad4
Showing 1 changed file with 11 additions and 10 deletions.
diff --git a/...tant/public/application/components/tabs/checkup/deprecations/reindex/flyout/container.tsx b/...tant/public/application/components/tabs/checkup/deprecations/reindex/flyout/container.tsx
@@ -9,16 +9,16 @@ import { DocLinksStart } from 'kibana/public';
 import { i18n } from '@kbn/i18n';
 import { FormattedMessage } from '@kbn/i18n/react';
 import {
+  EuiCallOut,
   EuiFlyout,
   EuiFlyoutHeader,
-  EuiPortal,
-  EuiTitle,
-  EuiCallOut,
   EuiLink,
+  EuiPortal,
   EuiSpacer,
+  EuiTitle,
 } from '@elastic/eui';
 
-import { EnrichedDeprecationInfo } from '../../../../../../../../common/types';
+import { EnrichedDeprecationInfo, ReindexStatus } from '../../../../../../../../common/types';
 
 import { ReindexState } from '../polling_service';
 import { ChecklistFlyoutStep } from './checklist_step';
@@ -117,13 +117,16 @@ export class ReindexFlyout extends React.Component<ReindexFlyoutProps, ReindexFl
     const { currentFlyoutStep } = this.state;
 
     let flyoutContents: React.ReactNode;
+
+    const globalCallout =
+      reindexBlocker === 'index-closed' && reindexState.status !== ReindexStatus.completed
+        ? getIndexClosedCallout(docLinks)
+        : undefined;
     switch (currentFlyoutStep) {
       case ReindexFlyoutStep.reindexWarnings:
         flyoutContents = (
           <WarningsFlyoutStep
-            renderGlobalCallouts={() =>
-              reindexBlocker === 'index-closed' ? getIndexClosedCallout(docLinks) : undefined
-            }
+            renderGlobalCallouts={() => globalCallout}
             closeFlyout={closeFlyout}
             warnings={reindexState.reindexWarnings!}
             advanceNextStep={this.advanceNextStep}
@@ -133,9 +136,7 @@ export class ReindexFlyout extends React.Component<ReindexFlyoutProps, ReindexFl
       case ReindexFlyoutStep.checklist:
         flyoutContents = (
           <ChecklistFlyoutStep
-            renderGlobalCallouts={() =>
-              reindexBlocker === 'index-closed' ? getIndexClosedCallout(docLinks) : undefined
-            }
+            renderGlobalCallouts={() => globalCallout}
             closeFlyout={closeFlyout}
             reindexState={reindexState}
             startReindex={startReindex}