address more feedback

packages/common/src/components/LoadingSpinner.tsx

@@ -0,0 +1,13 @@
+import React, { FC, PropsWithChildren } from 'react';
+
+import { Spinner, SpinnerProps } from '@patternfly/react-core';
+
+type LoadingSpinnerProps = PropsWithChildren &
+  SpinnerProps & {
+    isLoading: boolean;
+  };
+
+const LoadingSpinner: FC<LoadingSpinnerProps> = ({ isLoading, children, ...spinnerProps }) =>
+  isLoading ? <Spinner {...spinnerProps} /> : children;
+
+export default LoadingSpinner;

packages/common/src/components/index.ts

@@ -6,7 +6,7 @@ export * from './FormGroupWithHelpText';
 export * from './HelpIconPopover';
 export * from './Icons';
 export * from './LoadingDots';
-export * from './LoadingSpinner';
+export { default as LoadingSpinner } from './LoadingSpinner';
 export * from './Page';
 export * from './QueryClientHoc';
 export * from './TableView';

packages/forklift-console-plugin/src/modules/Plans/actions/PlanActionsDropdownItems.tsx

@@ -1,4 +1,4 @@
-import React, { useMemo } from 'react';
+import React from 'react';
 import { DropdownItemLink } from 'src/components/actions/DropdownItemLink';
 import { useModal } from 'src/modules/Providers/modals';
 import { getResourceUrl } from 'src/modules/Providers/utils/helpers';
@@ -16,6 +16,8 @@ import {
   PlanSummaryStatus,
 } from '../utils';
 
+import { getDuplicateActionDescription, getStartActionDescription } from './utils';
+
 export const PlanActionsDropdownItems = ({ data }: PlanActionsDropdownItemsProps) => {
   const { t } = useForkliftTranslation();
   const { showModal } = useModal();
@@ -53,40 +55,13 @@ export const PlanActionsDropdownItems = ({ data }: PlanActionsDropdownItemsProps
     showModal(<PlanDeleteModal resource={plan} model={PlanModel} />);
   };
 
-  const startActionDescription = useMemo(() => {
-    if (isPlanValidating) {
-      return t('The plan is being validated');
-    }
-
-    switch (status) {
-      case PlanSummaryStatus.Archived:
-        return t('Archived plans cannot be started');
-      case PlanSummaryStatus.Complete:
-        return t('All VMs were migrated');
-      case PlanSummaryStatus.Running:
-        return t('The plan is currently in progress');
-      case PlanSummaryStatus.CannotStart:
-        return t('The plan cannot be started');
-    }
-  }, [status, isPlanValidating]);
-
-  const duplicateActionDescription = React.useMemo(() => {
-    if (isPlanValidating) {
-      return t('The plan is being validated');
-    }
-
-    if (status === PlanSummaryStatus.CannotStart) {
-      return t('The plan cannot be duplicated');
-    }
-  }, [status, isPlanValidating]);
-
   return [
     <DropdownItem
       value={0}
       key="start"
       isDisabled={!canStart}
       onClick={onClickPlanStart}
-      description={startActionDescription}
+      description={getStartActionDescription(status, isPlanValidating, t)}
     >
       {buttonStartLabel}
     </DropdownItem>,
@@ -109,7 +84,7 @@ export const PlanActionsDropdownItems = ({ data }: PlanActionsDropdownItemsProps
         isPlanValidating
       }
       onClick={onClickDuplicate}
-      description={duplicateActionDescription}
+      description={getDuplicateActionDescription(status, isPlanValidating, t)}
     >
       {t('Duplicate Plan')}
     </DropdownItem>,

packages/forklift-console-plugin/src/modules/Plans/actions/utils.ts

@@ -0,0 +1,38 @@
+import { TFunction } from 'i18next';
+
+import { PlanSummaryStatus } from '../utils';
+
+export const getStartActionDescription = (
+  status: PlanSummaryStatus,
+  isPlanValidating: boolean,
+  t: TFunction,
+): string => {
+  if (isPlanValidating) {
+    return t('The plan is being validated');
+  }
+
+  switch (status) {
+    case PlanSummaryStatus.Archived:
+      return t('Archived plans cannot be started');
+    case PlanSummaryStatus.Complete:
+      return t('All VMs were migrated');
+    case PlanSummaryStatus.Running:
+      return t('The plan is currently in progress');
+    case PlanSummaryStatus.CannotStart:
+      return t('The plan cannot be started');
+  }
+};
+
+export const getDuplicateActionDescription = (
+  status: PlanSummaryStatus,
+  isPlanValidating: boolean,
+  t: TFunction,
+): string => {
+  if (isPlanValidating) {
+    return t('The plan is being validated');
+  }
+
+  if (status === PlanSummaryStatus.CannotStart) {
+    return t('The plan cannot be duplicated');
+  }
+};

packages/forklift-console-plugin/src/modules/Plans/utils/helpers/getConditionTypes.ts

@@ -1,17 +1,17 @@
 import { V1beta1Plan } from '@kubev2v/types';
 
-import { PlanConditionStatus } from '../types/PlanCondition';
+import { PlanConditionStatus, PlanConditionType } from '../types/PlanCondition';
 
 /**
  * Gets a record of plan types with truthful ('True') statuses
  * @param plan V1beta1Plan
  * @returns Record<string, boolean>
  */
-export const getConditionTypes = (plan: V1beta1Plan): Record<string, boolean> =>
+export const getConditionTypes = (plan: V1beta1Plan): Record<PlanConditionType, boolean> =>
   plan?.status?.conditions?.reduce((acc, condition) => {
     if (condition.status === PlanConditionStatus.True) {
       acc[condition.type] = true;
     }
 
     return acc;
-  }, {});
+  }, {} as Record<PlanConditionType, boolean>);