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
Jesse-Box pushed a commit that referenced this pull request Nov 12, 2025
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.
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
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.
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