Skip to content

fix(admin-omnibox): point Admin Panel link to users#2032

Open
jrf0110 wants to merge 1 commit intomainfrom
gt/toast/098c92ae
Open

fix(admin-omnibox): point Admin Panel link to users#2032
jrf0110 wants to merge 1 commit intomainfrom
gt/toast/098c92ae

Conversation

@jrf0110
Copy link
Copy Markdown
Contributor

@jrf0110 jrf0110 commented Apr 5, 2026

Summary

  • Point the admin omnibox Admin Panel shortcut at /admin/users so it lands on the main admin destination instead of the organizations page.

Verification

  • Not run (review-only change)

Visual Changes

N/A

Reviewer Notes

  • Single-line navigation fix in src/components/admin-omnibox/action-registry.ts.

{
label: 'Admin Panel',
href: '/admin/organizations',
href: '/admin/users',
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.

WARNING: Omnibox still has a second Admin Panel link pointing at /admin/organizations

This updates the registry action, but the persistent button in src/components/admin-omnibox/AdminOmnibox.tsx:177 still sends users to the organizations page. As written, the omnibox now has two Admin Panel entry points that land on different destinations, so the bug remains for anyone using the info-panel link.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot bot commented Apr 5, 2026

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

WARNING

File Line Issue
src/components/admin-omnibox/action-registry.ts 81 The registry link now points to /admin/users, but the omnibox info-panel button still links to /admin/organizations, so users can still land on the wrong admin destination.
Other Observations (not in diff)

None.

Files Reviewed (1 files)
  • src/components/admin-omnibox/action-registry.ts - 1 issue

Reviewed by gpt-5.4-20260305 · 192,495 tokens

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant