Skip to content

Commit

Permalink
[SecuritySolution] Link to the correct subscription upgrade "page/flo…
Browse files Browse the repository at this point in the history
…w" (elastic#142630)

* fix: link to the correct subscription upgrade "page/flow"

* test: fix tests

Co-authored-by: Kibana Machine <[email protected]>
  • Loading branch information
janmonschke and kibanamachine authored Oct 5, 2022
1 parent f644fde commit 25ba3c7
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 14 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ describe('Insights', () => {

expect(screen.getByTestId('related-alerts-by-ancestry')).toBeInTheDocument();
expect(
screen.queryByRole('link', { name: new RegExp(i18n.ALERT_UPSELL) })
screen.queryByRole('button', { name: new RegExp(i18n.INSIGHTS_UPSELL) })
).not.toBeInTheDocument();
});

Expand Down Expand Up @@ -179,7 +179,7 @@ describe('Insights', () => {
);

expect(
screen.getByRole('link', { name: new RegExp(i18n.ALERT_UPSELL) })
screen.getByRole('button', { name: new RegExp(i18n.INSIGHTS_UPSELL) })
).toBeInTheDocument();
expect(screen.queryByTestId('related-alerts-by-ancestry')).not.toBeInTheDocument();
});
Expand All @@ -198,7 +198,7 @@ describe('Insights', () => {

expect(screen.queryByTestId('related-alerts-by-ancestry')).not.toBeInTheDocument();
expect(
screen.queryByRole('link', { name: new RegExp(i18n.ALERT_UPSELL) })
screen.queryByRole('button', { name: new RegExp(i18n.INSIGHTS_UPSELL) })
).not.toBeInTheDocument();
});
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,12 @@
* 2.0.
*/

import React from 'react';
import React, { useCallback } from 'react';
import { EuiFlexGroup, EuiFlexItem, EuiLink, EuiIcon, EuiText } from '@elastic/eui';

import { euiStyled } from '@kbn/kibana-react-plugin/common';
import { ALERT_UPSELL } from './translations';
import { INSIGHTS_UPSELL } from './translations';
import { useNavigation } from '../../../lib/kibana';

const UpsellContainer = euiStyled.div`
border: 1px solid ${({ theme }) => theme.eui.euiColorLightShade};
Expand All @@ -22,6 +23,15 @@ const StyledIcon = euiStyled(EuiIcon)`
`;

export const RelatedAlertsUpsell = React.memo(() => {
const { getAppUrl, navigateTo } = useNavigation();
const subscriptionUrl = getAppUrl({
appId: 'management',
path: 'stack/license_management',
});
const goToSubscription = useCallback(() => {
navigateTo({ url: subscriptionUrl });
}, [navigateTo, subscriptionUrl]);

return (
<UpsellContainer>
<EuiFlexGroup alignItems="center" gutterSize="none">
Expand All @@ -30,13 +40,8 @@ export const RelatedAlertsUpsell = React.memo(() => {
</EuiFlexItem>
<EuiFlexItem>
<EuiText size="s">
<EuiLink
color="subdued"
href="https://www.elastic.co/pricing/"
rel="noopener noreferrer"
target="_blank"
>
{ALERT_UPSELL}
<EuiLink color="subdued" onClick={goToSubscription} target="_blank">
{INSIGHTS_UPSELL}
</EuiLink>
</EuiText>
</EuiFlexItem>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,9 +136,9 @@ export const SIMPLE_ALERT_TABLE_LIMITED = i18n.translate(
}
);

export const ALERT_UPSELL = i18n.translate(
export const INSIGHTS_UPSELL = i18n.translate(
'xpack.securitySolution.alertDetails.overview.insights.alertUpsellTitle',
{
defaultMessage: 'Get more insights with a subscription',
defaultMessage: 'Get more insights with a platinum subscription',
}
);

0 comments on commit 25ba3c7

Please sign in to comment.