Skip to content

Commit 77c8ddd

Browse files
malwilleygeorge-sentry
authored andcommitted
feat(aci): Add 'moved' badge to old alerts nav item (#112392)
1 parent 6e841c6 commit 77c8ddd

File tree

1 file changed

+31
-3
lines changed

1 file changed

+31
-3
lines changed

static/app/views/navigation/secondary/sections/issues/issuesSecondaryNavigation.tsx

Lines changed: 31 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,12 @@ import {Fragment} from 'react';
22
import {css} from '@emotion/react';
33
import styled from '@emotion/styled';
44

5-
import {t} from 'sentry/locale';
5+
import {Badge} from '@sentry/scraps/badge';
6+
import {Link} from '@sentry/scraps/link';
7+
import {Text} from '@sentry/scraps/text';
8+
import {Tooltip} from '@sentry/scraps/tooltip';
9+
10+
import {t, tct} from 'sentry/locale';
611
import {useOrganization} from 'sentry/utils/useOrganization';
712
import {makeAutomationBasePathname} from 'sentry/views/automations/pathnames';
813
import {ISSUE_TAXONOMY_CONFIG} from 'sentry/views/issueList/taxonomies';
@@ -108,11 +113,11 @@ function ConfigureSection({baseUrl}: {baseUrl: string}) {
108113
const {layout} = usePrimaryNavigation();
109114
const isSticky = layout === 'sidebar';
110115

116+
const hasWorkflowEngineUI = organization.features.includes('workflow-engine-ui');
111117
const hasRedirectOptOut = organization.features.includes(
112118
'workflow-engine-redirect-opt-out'
113119
);
114-
const shouldRedirectToWorkflowEngineUI =
115-
!hasRedirectOptOut && organization.features.includes('workflow-engine-ui');
120+
const shouldRedirectToWorkflowEngineUI = !hasRedirectOptOut && hasWorkflowEngineUI;
116121

117122
const alertsLink = shouldRedirectToWorkflowEngineUI
118123
? `${makeAutomationBasePathname(organization.slug)}?alertsRedirect=true`
@@ -133,6 +138,29 @@ function ConfigureSection({baseUrl}: {baseUrl: string}) {
133138
to={alertsLink}
134139
{...(!shouldRedirectToWorkflowEngineUI && {activeTo: `${baseUrl}/alerts/`})}
135140
analyticsItemName="issues_alerts"
141+
trailingItems={
142+
hasWorkflowEngineUI ? (
143+
<Tooltip
144+
isHoverable
145+
title={
146+
<Fragment>
147+
<Text as="p">{t('Alerts now live under Monitors.')}</Text>
148+
<Text as="p">
149+
{tct('See the [link:new Alerts page here.]', {
150+
link: (
151+
<Link
152+
to={`/organizations/${organization.slug}/monitors/alerts/`}
153+
/>
154+
),
155+
})}
156+
</Text>
157+
</Fragment>
158+
}
159+
>
160+
<Badge variant="muted">{t('Moved')}</Badge>
161+
</Tooltip>
162+
) : null
163+
}
136164
>
137165
{t('Alerts')}
138166
</SecondaryNavigation.Link>

0 commit comments

Comments
 (0)