Merge pull request #1421 from sgratch/set-plan-name-field-init-valida…

…tion-value-to-default Initilalize the validation value of the 'plan name' field to default instead of error
kubev2v · Jan 2, 2025 · f79044c · f79044c
2 parents b37b717 + 990f13e
commit f79044c
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,