Skip to content

Commit

Permalink
fix(insights): Fix "AI" link in new navigation (#86132)
Browse files Browse the repository at this point in the history
The "AI" view doesn't have a landing page. Instead, it should go
directly to the only module, which is "LLM Monitoring".
  • Loading branch information
gggritso authored Feb 28, 2025
1 parent 0bda995 commit 059f863
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion static/app/views/insights/navigation.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,9 @@ import {
} from 'sentry/views/insights/pages/mobile/settings';
import {DOMAIN_VIEW_BASE_URL} from 'sentry/views/insights/pages/settings';

import {MODULE_BASE_URLS} from './common/utils/useModuleURL';
import {ModuleName} from './types';

type InsightsNavigationProps = {
children: React.ReactNode;
};
Expand Down Expand Up @@ -60,7 +63,9 @@ export default function InsightsNavigation({children}: InsightsNavigationProps)
<SecondaryNav.Item to={`${baseUrl}/${MOBILE_LANDING_SUB_PATH}/`}>
{MOBILE_SIDEBAR_LABEL}
</SecondaryNav.Item>
<SecondaryNav.Item to={`${baseUrl}/${AI_LANDING_SUB_PATH}/`}>
<SecondaryNav.Item
to={`${baseUrl}/${AI_LANDING_SUB_PATH}/${MODULE_BASE_URLS[ModuleName.AI]}/`}
>
{AI_SIDEBAR_LABEL}
</SecondaryNav.Item>
</SecondaryNav.Section>
Expand Down

0 comments on commit 059f863

Please sign in to comment.