address pr comments

Signed-off-by: Derek Ho <dxho@amazon.com>

common/constants/shared.ts

@@ -52,9 +52,9 @@ export const observabilityPanelsID = 'observability-dashboards';
 export const observabilityPanelsTitle = 'Dashboards';
 export const observabilityPanelsPluginOrder = 5095;
 
-export const observabilityIntegrationsID = 'integrations';
-export const observabilityIntegrationsTitle = 'Integrations';
-export const observabilityIntegrationsPluginOrder = 9020;
+export const integrationsID = 'integrations';
+export const integrationsTitle = 'Integrations';
+export const integrationsPluginOrder = 9020;
 
 // Shared Constants
 export const SQL_DOCUMENTATION_URL = 'https://opensearch.org/docs/latest/search-plugins/sql/index/';

public/components/integrations/components/add_integration_flyout.tsx

@@ -6,8 +6,6 @@
 import _ from 'lodash';
 import {
   EuiButton,
-  EuiCallOut,
-  EuiCheckbox,
   EuiFieldText,
   EuiFlexGroup,
   EuiFlexItem,
@@ -17,16 +15,10 @@ import {
   EuiFlyoutHeader,
   EuiForm,
   EuiFormRow,
-  EuiLink,
-  EuiRadioGroup,
-  EuiSpacer,
-  EuiSuperSelect,
-  EuiText,
   EuiTitle,
 } from '@elastic/eui';
-import React, { Fragment, useState } from 'react';
+import React, { useState } from 'react';
 import { HttpStart } from '../../../../../../src/core/public';
-import { INTEGRATIONS_BASE } from '../../../../common/constants/shared';
 import { useToast } from '../../../../public/components/common/toast';
 
 interface IntegrationFlyoutProps {

public/components/integrations/components/available_integration_overview_page.tsx

@@ -9,23 +9,17 @@ import {
   EuiFilterButton,
   EuiFilterGroup,
   EuiFilterSelectItem,
-  EuiOverlayMask,
   EuiPage,
   EuiPageBody,
   EuiPopover,
   EuiPopoverTitle,
-  EuiSpacer,
-  EuiTab,
-  EuiTabs,
 } from '@elastic/eui';
 import _ from 'lodash';
-import React, { Dispatch, SetStateAction, useEffect, useState } from 'react';
-import { StringRegexOptions } from 'joi';
+import React, { useEffect, useState } from 'react';
 import { IntegrationHeader } from './integration_header';
 import { AvailableIntegrationsTable } from './available_integration_table';
 import { AvailableIntegrationsCardView } from './available_integration_card_view';
 import { INTEGRATIONS_BASE } from '../../../../common/constants/shared';
-import { getAddIntegrationModal } from './add_integration_modal';
 import { AvailableIntegrationOverviewPageProps } from './integration_types';
 import { useToast } from '../../../../public/components/common/toast';
 
@@ -44,7 +38,6 @@ export interface AvailableIntegrationType {
 export interface AvailableIntegrationsTableProps {
   loading: boolean;
   data: AvailableIntegrationsList;
-  showModal: (input: string) => void;
   isCardView: boolean;
   setCardView: (input: boolean) => void;
   renderCateogryFilters: () => React.JSX.Element;
@@ -56,7 +49,6 @@ export interface AvailableIntegrationsList {
 
 export interface AvailableIntegrationsCardViewProps {
   data: AvailableIntegrationsList;
-  showModal: (input: string) => void;
   isCardView: boolean;
   setCardView: (input: boolean) => void;
   query: string;
@@ -72,9 +64,6 @@ export function AvailableIntegrationOverviewPage(props: AvailableIntegrationOver
   const { setToast } = useToast();
   const [data, setData] = useState<AvailableIntegrationsList>({ hits: [] });
 
-  const [isModalVisible, setIsModalVisible] = useState(false);
-  const [modalLayout, setModalLayout] = useState(<EuiOverlayMask />);
-
   const [isPopoverOpen, setIsPopoverOpen] = useState(false);
 
   const onButtonClick = () => {
@@ -128,30 +117,6 @@ export function AvailableIntegrationOverviewPage(props: AvailableIntegrationOver
     </EuiFilterButton>
   );
 
-  const getModal = (name: string) => {
-    setModalLayout(
-      getAddIntegrationModal(
-        () => {
-          addIntegrationRequest(name);
-          setIsModalVisible(false);
-        },
-        () => {
-          setIsModalVisible(false);
-        },
-        'Name',
-        'Namespace',
-        'Tags (optional)',
-        name,
-        'prod',
-        'Add Integration Options',
-        'Cancel',
-        'Add',
-        'test'
-      )
-    );
-    setIsModalVisible(true);
-  };
-
   useEffect(() => {
     chrome.setBreadcrumbs([
       {
@@ -224,7 +189,6 @@ export function AvailableIntegrationOverviewPage(props: AvailableIntegrationOver
                   helper.every((compon) => hit.components.map((x) => x.name).includes(compon))
                 ),
               },
-              showModal: getModal,
               isCardView,
               setCardView,
               query,
@@ -238,13 +202,11 @@ export function AvailableIntegrationOverviewPage(props: AvailableIntegrationOver
                   helper.every((compon) => hit.components.map((x) => x.name).includes(compon))
                 ),
               },
-              showModal: getModal,
               isCardView,
               setCardView,
               renderCateogryFilters,
             })}
       </EuiPageBody>
-      {isModalVisible && modalLayout}
     </EuiPage>
   );
 }

public/components/integrations/components/integration.tsx

@@ -20,7 +20,6 @@ import { IntegrationOverview } from './integration_overview_panel';
 import { IntegrationDetails } from './integration_details_panel';
 import { IntegrationFields } from './integration_fields_panel';
 import { IntegrationAssets } from './integration_assets_panel';
-import { getAddIntegrationModal } from './add_integration_modal';
 import { AvailableIntegrationProps } from './integration_types';
 import { INTEGRATIONS_BASE } from '../../../../common/constants/shared';
 import { IntegrationScreenshots } from './integration_screenshots_panel';