Merge pull request #1614 from bipuladh/fix-provider-mode

Reset radio button when Provider mode is selected from dropdown
red-hat-storage · Oct 8, 2024 · 6f69f2e · 6f69f2e
2 parents 79087f7 + c27a9c5
commit 6f69f2e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/...-storage-system/create-storage-system-steps/backing-storage-step/backing-storage-step.tsx b/...-storage-system/create-storage-system-steps/backing-storage-step/backing-storage-step.tsx
@@ -285,7 +285,7 @@ export const BackingStorage: React.FC<BackingStorageProps> = ({
         },
       });
     }
-  }, [dispatch, allowedExternalStorage.length, hasOCS]);
+  }, [dispatch, allowedExternalStorage.length, hasOCS, isProviderModePresent]);
 
   React.useEffect(() => {
     /*

diff --git a/...ate-storage-system/create-storage-system-steps/backing-storage-step/select-deployment.tsx b/...ate-storage-system/create-storage-system-steps/backing-storage-step/select-deployment.tsx
@@ -1,6 +1,6 @@
 import * as React from 'react';
 import { FDF_FLAG } from '@odf/core/redux';
-import { DeploymentType } from '@odf/core/types';
+import { BackingStorageType, DeploymentType } from '@odf/core/types';
 import { useCustomTranslation } from '@odf/shared/useCustomTranslationHook';
 import { useFlag } from '@openshift-console/dynamic-plugin-sdk';
 import {
@@ -49,6 +49,12 @@ export const SelectDeployment: React.FC<SelectDeploymentProps> = ({
       // 'value' on SelectProps['onSelect'] is string hence does not match with payload of type "DeploymentType"
       payload: value as DeploymentType,
     });
+    if (value === DeploymentType.PROVIDER_MODE) {
+      dispatch({
+        type: 'backingStorage/setType',
+        payload: BackingStorageType.EXISTING,
+      });
+    }
     setIsSelectOpen(false);
   };