Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(site): show AI settings and AI Sessions in mobile admin menu - #27191

Merged
tracyjohnsonux merged 8 commits into
mainfrom
tj/mobile-menu-ai-settings
Jul 13, 2026
Merged

fix(site): show AI settings and AI Sessions in mobile admin menu#27191
tracyjohnsonux merged 8 commits into
mainfrom
tj/mobile-menu-ai-settings

Conversation

@tracyjohnsonux

@tracyjohnsonux tracyjohnsonux commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Problem

The desktop DeploymentDropdown surfaces AI and AI sessions items under Admin settings (behind canViewAISettings / canViewAIBridge), but the mobile MobileMenu was never updated to match — its AdminSettingsSub only knows about Deployment, Organizations, Audit logs, Connection logs, and Healthcheck.

This is a leftover from #25582 ("promote AI settings to a top-level section"), which threaded canViewAISettings through Navbar / NavbarView / DeploymentDropdown but did not touch MobileMenu.tsx. canViewAIBridge has the same oversight.

Fix

  • Extend MobileMenuPermissions with canViewAIBridge and canViewAISettings.
  • Render AI (→ /ai/settings) and AI Sessions (→ /ai-gateway/sessions) in the mobile Admin settings collapsible, in the same order as the desktop dropdown.
  • Thread the two flags through NavbarView into MobileMenu.
  • Cover the new args in MobileMenu.stories.tsx (Admin story now shows the AI items; Auditor / OrgAdmin / Member keep them hidden).

No backend, permission, or routing changes.

🤖 This PR was drafted by Coder Agents on behalf of @tracyjohnsonux and needs a human review.

@linear-code

linear-code Bot commented Jul 13, 2026

Copy link
Copy Markdown

CODAGT-793

{canViewAuditLog && (
<DropdownMenuItem asChild>
<Link to={linkToAuditing}>Audit Logs</Link>
<Link to={linkToAuditing}>Audit logs</Link>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just checking we want these capitalization changes?

@tracyjohnsonux tracyjohnsonux Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yep, the've been wrong for a while and make my eye twitch every time I see them. Eventually, we'll get the agents catching these items.

@tracyjohnsonux
tracyjohnsonux merged commit 043d367 into main Jul 13, 2026
25 of 27 checks passed
@tracyjohnsonux
tracyjohnsonux deleted the tj/mobile-menu-ai-settings branch July 13, 2026 22:24
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants