Initilalize the validation value for the wizard's 'plan name' field t…

…o default instead of error Reference: https://issues.redhat.com/browse/MTV-1651 To avoid displaying an error on initialization for the 'plan name'empty field, when entering the wizard's step 2, leave the initialized validation value for this field as 'default. Signed-off-by: Sharon Gratch <[email protected]>
kubev2v · Dec 26, 2024 · 990f13e · 990f13e
1 parent e35e84a
commit 990f13e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/packages/forklift-console-plugin/src/modules/Plans/views/create/PlanCreatePage.tsx b/packages/forklift-console-plugin/src/modules/Plans/views/create/PlanCreatePage.tsx
@@ -90,7 +90,8 @@ export const PlanCreatePage: React.FC<{ namespace: string }> = ({ namespace }) =
         !emptyContext &&
         !(
           !!state?.flow?.apiError ||
-          Object.values(state?.validation || []).some((validation) => validation === 'error')
+          Object.values(state?.validation || []).some((validation) => validation === 'error') ||
+          state?.validation?.planName === 'default'
         ),
       canJumpTo: filterState?.selectedVMs?.length > 0,
       nextButtonText: 'Create migration plan',

diff --git a/packages/forklift-console-plugin/src/modules/Providers/views/migrate/reducer/reducer.ts b/packages/forklift-console-plugin/src/modules/Providers/views/migrate/reducer/reducer.ts
@@ -137,10 +137,6 @@ const handlers: {
   ) {
     // triggered from useEffect on any data change
     draft.existingResources.plans = existingPlans;
-    draft.validation.planName = validatePlanName(
-      draft.underConstruction.plan.metadata.name,
-      existingPlans,
-    );
   },
   [SET_AVAILABLE_TARGET_NAMESPACES](
     draft,