Remove 'Tech Preview' labels from EA workflows

elastic · May 3, 2023 · b9a59fe · b9a59fe
1 parent a16930f
commit b9a59fe
Show file tree

Hide file tree

Showing 9 changed files with 19 additions and 38 deletions.
diff --git a/...ins/security_solution/public/common/components/event_details/cti_details/risk_summary.tsx b/...ins/security_solution/public/common/components/event_details/cti_details/risk_summary.tsx
@@ -48,6 +48,13 @@ const RiskSummaryComponent: React.FC<RiskEntity> = ({ risk, riskEntity, original
               riskScoreEntity={riskEntity}
             />
           }
+          toolTipTitle={
+            <RiskScoreHeaderTitle
+              title={i18n.RISK_DATA_TITLE(riskEntity)}
+              riskScoreEntity={riskEntity}
+              showTechnicalPreviewBadge
+            />
+          }
           toolTipContent={
             <FormattedMessage
               id="xpack.securitySolution.alertDetails.overview.riskDataTooltipContent"

diff --git a/...urity_solution/public/common/components/event_details/cti_details/threat_summary_view.tsx b/...urity_solution/public/common/components/event_details/cti_details/threat_summary_view.tsx
@@ -79,7 +79,8 @@ export const EnrichedDataRow: React.FC<{
 export const ThreatSummaryPanelHeader: React.FC<{
   title: string | React.ReactNode;
   toolTipContent: React.ReactNode;
-}> = ({ title, toolTipContent }) => {
+  toolTipTitle?: React.ReactNode;
+}> = ({ title, toolTipContent, toolTipTitle }) => {
   const [isPopoverOpen, setIsPopoverOpen] = useState(false);
 
   const onClick = useCallback(() => {
@@ -111,7 +112,7 @@ export const ThreatSummaryPanelHeader: React.FC<{
             />
           }
         >
-          <EuiPopoverTitle>{title}</EuiPopoverTitle>
+          <EuiPopoverTitle>{toolTipTitle ?? title}</EuiPopoverTitle>
           <EuiText size="s" style={{ width: '270px' }}>
             {toolTipContent}
           </EuiText>

diff --git a/...lugins/security_solution/public/explore/components/risk_score/enable_risk_score/index.tsx b/...lugins/security_solution/public/explore/components/risk_score/enable_risk_score/index.tsx
@@ -49,7 +49,10 @@ const EnableRiskScoreComponent = ({
 
   return (
     <EuiPanel hasBorder>
-      <HeaderSection title={<RiskScoreHeaderTitle riskScoreEntity={entityType} />} titleSize="s" />
+      <HeaderSection
+        title={<RiskScoreHeaderTitle riskScoreEntity={entityType} showTechnicalPreviewBadge />}
+        titleSize="s"
+      />
       <EuiEmptyPrompt
         title={<h2>{text.cta}</h2>}
         body={

diff --git a/...on/public/explore/components/risk_score/risk_score_onboarding/risk_score_header_title.tsx b/...on/public/explore/components/risk_score/risk_score_onboarding/risk_score_header_title.tsx
@@ -14,15 +14,19 @@ import { TECHNICAL_PREVIEW } from './translations';
 const RiskScoreHeaderTitleComponent = ({
   riskScoreEntity,
   title,
+  showTechnicalPreviewBadge = false,
 }: {
   riskScoreEntity: RiskScoreEntity;
   title?: string;
+  showTechnicalPreviewBadge?: boolean;
 }) => {
   return (
     <>
       {title ??
         (riskScoreEntity === RiskScoreEntity.user ? i18n.USER_RISK_TITLE : i18n.HOST_RISK_TITLE)}
-      <NavItemBetaBadge text={TECHNICAL_PREVIEW} className="eui-alignMiddle" />
+      {showTechnicalPreviewBadge && (
+        <NavItemBetaBadge text={TECHNICAL_PREVIEW} className="eui-alignMiddle" />
+      )}
     </>
   );
 };

diff --git a/x-pack/plugins/security_solution/public/explore/hosts/pages/details/nav_tabs.test.tsx b/x-pack/plugins/security_solution/public/explore/hosts/pages/details/nav_tabs.test.tsx
@@ -81,18 +81,4 @@ describe('navTabsHostDetails', () => {
 
     expect(sessionsTab).not.toBeTruthy();
   });
-
-  test('it should display Beta badge for risk tab', () => {
-    const tabs = navTabsHostDetails({
-      hasMlUserPermissions: false,
-      isRiskyHostsEnabled: true,
-      hostName: mockHostName,
-    });
-
-    const riskTab = Object.values<TabNavigationItemProps>(tabs).find(
-      (item) => item.id === HostsTableType.risk
-    );
-
-    expect(riskTab?.isBeta).toEqual(true);
-  });
 });
diff --git a/x-pack/plugins/security_solution/public/explore/hosts/pages/details/nav_tabs.tsx b/x-pack/plugins/security_solution/public/explore/hosts/pages/details/nav_tabs.tsx
@@ -10,7 +10,6 @@ import * as i18n from '../translations';
 import type { HostDetailsNavTab } from './types';
 import { HostsTableType } from '../../store/model';
 import { HOSTS_PATH } from '../../../../../common/constants';
-import { TECHNICAL_PREVIEW } from '../../../../overview/pages/translations';
 
 const getTabsOnHostDetailsUrl = (hostName: string, tabName: HostsTableType) =>
   `${HOSTS_PATH}/name/${hostName}/${tabName}`;
@@ -58,10 +57,6 @@ export const navTabsHostDetails = ({
       name: i18n.NAVIGATION_HOST_RISK_TITLE,
       href: getTabsOnHostDetailsUrl(hostName, HostsTableType.risk),
       disabled: false,
-      isBeta: true,
-      betaOptions: {
-        text: TECHNICAL_PREVIEW,
-      },
     },
     [HostsTableType.sessions]: {
       id: HostsTableType.sessions,

diff --git a/x-pack/plugins/security_solution/public/explore/hosts/pages/nav_tabs.tsx b/x-pack/plugins/security_solution/public/explore/hosts/pages/nav_tabs.tsx
@@ -10,7 +10,6 @@ import * as i18n from './translations';
 import { HostsTableType } from '../store/model';
 import type { HostsNavTab } from './navigation/types';
 import { HOSTS_PATH } from '../../../../common/constants';
-import { TECHNICAL_PREVIEW } from '../../../overview/pages/translations';
 
 const getTabsOnHostsUrl = (tabName: HostsTableType) => `${HOSTS_PATH}/${tabName}`;
 
@@ -54,10 +53,6 @@ export const navTabsHosts = ({
       name: i18n.NAVIGATION_HOST_RISK_TITLE,
       href: getTabsOnHostsUrl(HostsTableType.risk),
       disabled: false,
-      isBeta: true,
-      betaOptions: {
-        text: TECHNICAL_PREVIEW,
-      },
     },
     [HostsTableType.sessions]: {
       id: HostsTableType.sessions,

diff --git a/x-pack/plugins/security_solution/public/explore/users/pages/details/nav_tabs.tsx b/x-pack/plugins/security_solution/public/explore/users/pages/details/nav_tabs.tsx
@@ -10,7 +10,6 @@ import * as i18n from '../translations';
 import type { UsersDetailsNavTab } from './types';
 import { UsersTableType } from '../../store/model';
 import { USERS_PATH } from '../../../../../common/constants';
-import { TECHNICAL_PREVIEW } from '../../../../overview/pages/translations';
 
 const getTabsOnUsersDetailsUrl = (userName: string, tabName: UsersTableType) =>
   `${USERS_PATH}/name/${userName}/${tabName}`;
@@ -46,10 +45,6 @@ export const navTabsUsersDetails = (
       name: i18n.NAVIGATION_RISK_TITLE,
       href: getTabsOnUsersDetailsUrl(userName, UsersTableType.risk),
       disabled: false,
-      isBeta: true,
-      betaOptions: {
-        text: TECHNICAL_PREVIEW,
-      },
     },
   };
 

diff --git a/x-pack/plugins/security_solution/public/explore/users/pages/nav_tabs.tsx b/x-pack/plugins/security_solution/public/explore/users/pages/nav_tabs.tsx
@@ -10,7 +10,6 @@ import * as i18n from './translations';
 import { UsersTableType } from '../store/model';
 import type { UsersNavTab } from './navigation/types';
 import { USERS_PATH } from '../../../../common/constants';
-import { TECHNICAL_PREVIEW } from '../../../overview/pages/translations';
 
 const getTabsOnUsersUrl = (tabName: UsersTableType) => `${USERS_PATH}/${tabName}`;
 
@@ -50,10 +49,6 @@ export const navTabsUsers = (
       name: i18n.NAVIGATION_RISK_TITLE,
       href: getTabsOnUsersUrl(UsersTableType.risk),
       disabled: false,
-      isBeta: true,
-      betaOptions: {
-        text: TECHNICAL_PREVIEW,
-      },
     },
   };