-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix: improve AI Bridge request logs UI/UX #21252
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Show resolved
Hide resolved
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Outdated
Show resolved
Hide resolved
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Outdated
Show resolved
Hide resolved
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Outdated
Show resolved
Hide resolved
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Outdated
Show resolved
Hide resolved
…uestLogsRow.tsx Co-authored-by: ケイラ <mckayla@hey.com>
📚 Documentation Check✨ Suggested Enhancement (Optional)The UI improvements in this PR (provider icons, token badges, improved layout) could benefit the user-facing documentation:
This would help users understand they can monitor AI Bridge usage directly in Coder's UI, not just through Grafana. Note: This is purely a visual enhancement to existing functionality - no new features or behavior changes, so documentation updates are optional. This comment was generated by an AI Agent through Coder Tasks |
|
All contributors have signed the CLA ✍️ ✅ |
0a75b0c to
4941f85
Compare
Addressing feedback found in #21252 > [!IMPORTANT] > This pull-request removes endpoints from `ExperimentalHandler` from `coderd.go` and promotes the endpoints within the frontend. This means that we will no longer be serving AI Bridge under the `/api/experimental/` prefix now that things reached release in [`v2.29.0`](https://github.com/coder/coder/releases/tag/v2.29.0). > > ### Migration > > The `/api/experimental/aibridge` prefix has been removed. Any clients, scripts, or integrations that previously called AI Bridge endpoints under `/api/experimental/aibridge` must be updated to use the `/api/v2/aibridge` stable API routes introduced in v2.29.0. | Position | Pull-request | | -------- | ------------ | | | [fix: improve AI Bridge request logs UI/UX](#21252) | | | [feat: add AI Bridge request logs model filter](#21259) | | ✅ | [fix: promote AIBridge from `ExperimentalHandler`](#21278) | --------- Co-authored-by: Susana Ferreira <susana@coder.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hate to keep blocking this over icons, but do we really need a second claude svg? 😅
Closes #21052 and is related to #21218
This pull request implements a bunch of changes the team requested to improve the overall user experience when navigating to the Request Logs page. Since these are mostly UI changes, screenshots do a much better job of showing what's different than me trying to explain everything in words, so I've included them below.
Warning
More globally we've changed how we're rendering
<PaginationHeader />, therefore we've renamed it to<PaginationAmount />. This will effect multiple layouts however its a global improvement we're intentionally looking to make.Preview
Changes
MMM(i.eDec).namefield falling back on.username.text-xstotext-smto increase overall legibility globally.<Badge />and gives longer names a chance to render.<Badge />alongside their icons.<Badge />-group alongside truncation (when necessary).5is rendered with the proper name being visible on hover (for use if truncated).Showing X of Xcounts to the bottom of the tables (effects other components outside of AI Bridge).Providerfilter.<dt>element with the badgeExperimentalHandler