Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions knip.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,9 @@ const productionEntryPoints = [
'static/app/components/core/quote/*.tsx',
// Prevent exception until we build out coverage
'static/app/components/prevent/virtualRenderers/**/*.{js,ts,tsx}',
// Temporary Prevent TA exceptions until the code is removed
'static/app/views/nav/secondary/sections/prevent/**/*.{js,ts,tsx}',
'static/app/views/prevent/**/*.{js,ts,tsx}',
// todo we currently keep all icons
'static/app/icons/**/*.{js,mjs,ts,tsx}',
// todo find out how chartcuterie works
Expand Down
37 changes: 0 additions & 37 deletions static/app/router/routes.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2358,42 +2358,6 @@ function buildRoutes(): RouteObject[] {
children: exploreChildren,
};

const preventRoutes: SentryRouteObject = {
path: '/prevent/',
withOrgPath: true,
component: make(() => import('sentry/views/prevent/index')),
children: [
{
index: true,
redirectTo: 'tests/',
},
{
path: 'tests/',
component: make(() => import('sentry/views/prevent/tests/testsWrapper')),
children: [
{
index: true,
component: make(() => import('sentry/views/prevent/tests/tests')),
},
{
path: 'new/',
component: make(() => import('sentry/views/prevent/tests/onboarding')),
},
],
},
{
path: 'tokens/',
component: make(() => import('sentry/views/prevent/tokens/tokensWrapper')),
children: [
{
index: true,
component: make(() => import('sentry/views/prevent/tokens/tokens')),
},
],
},
],
};

const preprodChildren: SentryRouteObject[] = [
{
index: true,
Expand Down Expand Up @@ -2828,7 +2792,6 @@ function buildRoutes(): RouteObject[] {
issueRoutes,
alertRoutes,
monitorRoutes,
preventRoutes,
preprodRoutes,
pullRequestRoutes,
replayRoutes,
Expand Down
7 changes: 3 additions & 4 deletions static/app/views/nav/primary/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ import {
IconSettings,
IconSiren,
} from 'sentry/icons';
import type {Organization} from 'sentry/types/organization';
import useOrganization from 'sentry/utils/useOrganization';
import {getDefaultExploreRoute} from 'sentry/views/explore/utils';
import {useNavContext} from 'sentry/views/nav/context';
Expand Down Expand Up @@ -69,11 +68,11 @@ function SidebarFooter({children}: {children: React.ReactNode}) {
);
}

function showPreventNav(organization: Organization) {
function showPreventNav() {
// only people with test analytics can see the prevent nav
// Legacy Seer and New Seer orgs are getting a Seer Config Reminder icon, which
// means that the only Prevent sub-nav item remaining is the Tests item.
return organization.features.includes('prevent-test-analytics');
return false;
}

export function PrimaryNavigationItems() {
Expand Down Expand Up @@ -161,7 +160,7 @@ export function PrimaryNavigationItems() {
</NavTourElement>
</Feature>

{showPreventNav(organization) ? (
{showPreventNav() ? (
<Container position="relative" height="100%">
<SidebarLink
to={`/${prefix}/prevent/tests/`}
Expand Down
3 changes: 1 addition & 2 deletions static/app/views/nav/secondary/secondaryNavContent.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import {ExploreSecondaryNav} from 'sentry/views/nav/secondary/sections/explore/e
import {InsightsSecondaryNav} from 'sentry/views/nav/secondary/sections/insights/insightsSecondaryNav';
import {IssuesSecondaryNav} from 'sentry/views/nav/secondary/sections/issues/issuesSecondaryNav';
import {MonitorsSecondaryNav} from 'sentry/views/nav/secondary/sections/monitors/monitorsSecondaryNav';
import PreventSecondaryNav from 'sentry/views/nav/secondary/sections/prevent/preventSecondaryNav';
import {SettingsSecondaryNav} from 'sentry/views/nav/secondary/sections/settings/settingsSecondaryNav';
import {PrimaryNavGroup} from 'sentry/views/nav/types';

Expand All @@ -24,7 +23,7 @@ export function SecondaryNavContent({group}: {group: PrimaryNavGroup}): ReactNod
case PrimaryNavGroup.MONITORS:
return <MonitorsSecondaryNav />;
case PrimaryNavGroup.PREVENT:
return <PreventSecondaryNav />;
return null;
case PrimaryNavGroup.SETTINGS:
return <SettingsSecondaryNav />;
case PrimaryNavGroup.ADMIN:
Expand Down

This file was deleted.

Loading