addressed PR comments

Signed-off-by: Amardeepsingh Siglani <[email protected]>
opensearch-project · Oct 30, 2024 · 8d7f5e5 · 8d7f5e5
1 parent 93ae981
commit 8d7f5e5
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 10 deletions.
diff --git a/src/plugins/data/common/datasets/types.ts b/src/plugins/data/common/datasets/types.ts
@@ -258,21 +258,26 @@ export interface DatasetField {
   // TODO:  osdFieldType?
 }
 
-export type CanvasBannerProps =
+export enum DatasetCanvasBannerComponentType {
+  Callout = 'callout',
+  Custom = 'custom',
+}
+
+export type DatasetCanvasBannerProps =
   | {
-      componentType: 'callout';
+      componentType: DatasetCanvasBannerComponentType.Callout;
       title: string;
       iconType?: string;
       content?: React.ReactNode;
     }
   | {
-      componentType: 'callout';
+      componentType: DatasetCanvasBannerComponentType.Callout;
       title?: string;
       iconType?: string;
       content: React.ReactNode;
     }
   | {
-      componentType: 'custom';
+      componentType: DatasetCanvasBannerComponentType.Custom;
       content: React.ReactNode;
     };
 

diff --git a/src/plugins/data/public/ui/dataset_selector/configurator.tsx b/src/plugins/data/public/ui/dataset_selector/configurator.tsx
@@ -57,7 +57,7 @@ export const Configurator = ({
   const indexedViewsService = type?.indexedViewsService;
   const [selectedIndexedView, setSelectedIndexedView] = useState<string | undefined>();
   const [indexedViews, setIndexedViews] = useState<IndexedView[]>([]);
-  const [loadingIndexedViews, setLoadingIndexedViews] = useState(false);
+  const [isLoadingIndexedViews, setIsLoadingIndexedViews] = useState(false);
 
   useEffect(() => {
     setLanguages(type?.supportedLanguages(dataset) || []);
@@ -75,9 +75,9 @@ export const Configurator = ({
   useEffect(() => {
     const getIndexedViews = async () => {
       if (indexedViewsService) {
-        setLoadingIndexedViews(true);
+        setIsLoadingIndexedViews(true);
         const fetchedIndexedViews = await indexedViewsService.getIndexedViews(baseDataset);
-        setLoadingIndexedViews(false);
+        setIsLoadingIndexedViews(false);
         setIndexedViews(fetchedIndexedViews || []);
       }
     };
@@ -87,7 +87,7 @@ export const Configurator = ({
 
   const submitDisabled = useMemo(() => {
     return (
-      loadingIndexedViews ||
+      isLoadingIndexedViews ||
       (timeFieldName === undefined &&
         !(
           languageService.getLanguage(selectedLanguage)?.hideDatePicker ||
@@ -96,7 +96,14 @@ export const Configurator = ({
         timeFields &&
         timeFields.length > 0)
     );
-  }, [dataset, selectedLanguage, timeFieldName, timeFields, languageService, loadingIndexedViews]);
+  }, [
+    dataset,
+    selectedLanguage,
+    timeFieldName,
+    timeFields,
+    languageService,
+    isLoadingIndexedViews,
+  ]);
 
   useEffect(() => {
     const fetchFields = async () => {
@@ -165,7 +172,7 @@ export const Configurator = ({
               )}
             >
               <EuiSelect
-                isLoading={loadingIndexedViews}
+                isLoading={isLoadingIndexedViews}
                 options={indexedViews.map(({ name }) => ({
                   text: name,
                   value: name,