Skip to content

Conversation

@markstory
Copy link
Member

The admin overview page is empty now. Instead of the default view of /manage/ being an empty view we can show the environment page instead.

The admin overview page is empty now. Instead of the default view of
`/manage/` being an empty view we can show the environment page instead.
@markstory markstory requested a review from aldy505 November 7, 2025 21:01
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 7, 2025
{
index: true,
component: make(() => import('sentry/views/admin/adminOverview')),
component: make(() => import('sentry/views/admin/adminEnvironment')),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Routing Redundancy Impairs Navigation Consistency.

The index route at /manage/ now renders adminEnvironment, but there's also an explicit route at /manage/status/environment/ that renders the same component. This creates duplicate routes for the same content and causes navigation inconsistency—when viewing /manage/, the "Environment" navigation item won't highlight as active since it expects /manage/status/environment/.

Fix in Cursor Fix in Web

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we fix this?

Copy link
Collaborator

@aldy505 aldy505 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have write access 😅

@markstory markstory requested a review from a team November 10, 2025 16:58
@markstory markstory merged commit f465972 into master Nov 10, 2025
48 checks passed
@markstory markstory deleted the manage-overview branch November 10, 2025 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants