Skip to content

Feat: (UI) running model add Try to API and add useMenuAuth hook - #5098

Merged
qinxuye merged 14 commits into
xorbitsai:mainfrom
maoyuehui:feat-new-ui-version-7
Jul 2, 2026
Merged

qinxuye merged 14 commits into
xorbitsai:mainfrom
maoyuehui:feat-new-ui-version-7

Conversation

@maoyuehui

Copy link
Copy Markdown
Collaborator
  1. running model add Try to API
  2. add useMenuAuth hook
  3. part style fix

@XprobeBot XprobeBot added this to the v2.x milestone Jun 29, 2026
@maoyuehui
maoyuehui requested review from qinxuye and yiboyasss June 29, 2026 03:29

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request introduces a 'Try To API' feature that allows users to view and copy API code examples in multiple languages (Python, TypeScript, Java, Go, and Shell) for various model capabilities. It also refactors permission checks across the sidebar and API key management pages using a new centralized useMenuAuth hook. The review feedback highlights several key improvements: correcting a permission mapping mismatch in API key management, resolving potential compilation and runtime errors in the generated Go and TypeScript code examples, ensuring consistency in the generated curl commands by including optional fields, and fixing a TypeScript typing issue in the model detail transformation utility.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread frontend/src/components/pages/api-key-management/index.tsx Outdated
Comment thread frontend/src/components/pages/running-model-detail/utils.ts
maoyuehui

This comment was marked as resolved.

@maoyuehui maoyuehui changed the title Feat: (UI) running model add Try to API Feat: (UI) running model add Try to API and add useMenuAuth hook Jun 29, 2026

@qinxuye qinxuye left a comment

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.

Review comments for PR 5098.

Comment thread frontend/src/components/layout/sidebar.tsx Outdated
Comment thread frontend/src/components/pages/api-key-management/index.tsx Outdated
Comment thread frontend/src/components/pages/running-model-detail/components/try-api-drawer.tsx Outdated
Comment thread frontend/src/components/pages/running-model-detail/components/try-api-drawer.tsx Outdated
Comment thread frontend/src/components/layout/sidebar.tsx Outdated
Comment thread frontend/src/components/layout/sidebar.tsx Outdated
Comment thread frontend/src/components/pages/api-key-management/index.tsx Outdated
Comment thread frontend/src/components/pages/api-key-management/index.tsx Outdated
Comment thread frontend/src/components/pages/running-model-detail/components/try-api-drawer.tsx Outdated
Comment thread frontend/src/components/layout/sidebar.tsx Outdated
Comment thread frontend/src/components/layout/sidebar.tsx Outdated
Comment thread frontend/src/components/pages/api-key-management/index.tsx Outdated
Comment thread frontend/src/components/pages/api-key-management/index.tsx Outdated
Comment thread frontend/src/components/pages/running-model-detail/components/try-api-drawer.tsx Outdated
Comment thread frontend/src/components/layout/sidebar.tsx Outdated
Comment thread frontend/src/components/pages/api-key-management/index.tsx Outdated
Comment thread frontend/src/components/pages/api-key-management/index.tsx Outdated
Comment thread frontend/src/components/pages/running-model-detail/components/try-api-drawer.tsx Outdated

@qinxuye qinxuye left a comment

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.

LGTM

@qinxuye
qinxuye merged commit 113974d into xorbitsai:main Jul 2, 2026
8 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants