Skip to content

feat(seer): Show the Seer settings link even when hideAiFeatures is enabled#112510

Merged
ryan953 merged 1 commit intomasterfrom
ryan953/seer-hideAiFeatures-nav
Apr 8, 2026
Merged

feat(seer): Show the Seer settings link even when hideAiFeatures is enabled#112510
ryan953 merged 1 commit intomasterfrom
ryan953/seer-hideAiFeatures-nav

Conversation

@ryan953
Copy link
Copy Markdown
Member

@ryan953 ryan953 commented Apr 8, 2026

When hideAiFeatures is enabled Seer is disabled, that's always the way. hideAiFeatures can be quickly toggled from the Settings > General page.

What's confusing though is that we also hide the button to Seer in the sidenav. This gave me a freight because i didn't see the Seer button and didn't know why. One of the tenets we've had for a long time is Disable and Explain, and we already had a message to explain things.

Now we keep the sidebar button, and show the explaination.

SCR-20260408-kpjo

@ryan953 ryan953 requested a review from a team as a code owner April 8, 2026 19:53
@ryan953 ryan953 requested a review from a team April 8, 2026 19:53
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 8, 2026
@ryan953 ryan953 requested a review from a team April 8, 2026 19:53
@ryan953 ryan953 enabled auto-merge (squash) April 8, 2026 19:53
@ryan953 ryan953 merged commit 5d69a6f into master Apr 8, 2026
66 checks passed
@ryan953 ryan953 deleted the ryan953/seer-hideAiFeatures-nav branch April 8, 2026 20:00
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
…nabled (#112510)

When `hideAiFeatures` is enabled Seer is disabled, that's always the
way. `hideAiFeatures` can be quickly toggled from the Settings > General
page.

What's confusing though is that we also hide the button to Seer in the
sidenav. This gave me a freight because i didn't see the Seer button and
didn't know why. One of the tenets we've had for a long time is [Disable
and
Explain](https://develop.sentry.dev/frontend/design-tenets/#disable-and-explain-dont-hide),
and we already had a message to explain things.

Now we keep the sidebar button, and show the explaination.

<img width="1256" height="388" alt="SCR-20260408-kpjo"
src="https://github.com/user-attachments/assets/d53557bf-d2e2-453c-a506-ebfdfaf2b608"
/>
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.

2 participants