-
Couldn't load subscription status.
- Fork 78
feat(FR-1531): add active agent list to admin dashboard page #4365
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
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has required the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 51.38% | 130/253 |
| 🔴 | Branches | 30.3% | 80/264 |
| 🔴 | Functions | 34.48% | 20/58 |
| 🔴 | Lines | 53.85% | 119/221 |
Test suite run success
55 tests passing in 3 suites.
Report generated by 🧪jest coverage report action from 7119a62
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 4.67% (+0.01% 🔼) |
532/11395 |
| 🔴 | Branches | 3.78% (+0.02% 🔼) |
302/7983 |
| 🔴 | Functions | 2.88% (+0.01% 🔼) |
102/3537 |
| 🔴 | Lines | 4.62% (+0.01% 🔼) |
514/11136 |
Test suite run success
121 tests passing in 14 suites.
Report generated by 🧪jest coverage report action from 7119a62
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.
49dd79c to
7bcecef
Compare
09b8426 to
01dee45
Compare
7bcecef to
3dc3b79
Compare
3dc3b79 to
f70c2d9
Compare
|
I modified it also. please check again :) |
01dee45 to
9c43d1d
Compare
f70c2d9 to
412ed7b
Compare
412ed7b to
753ed25
Compare
9c43d1d to
4e1ef17
Compare
753ed25 to
d4c0c39
Compare
4e1ef17 to
c99d1cb
Compare
2d0d111 to
9c1cf1b
Compare
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.
Pull Request Overview
This PR adds a new "Active Agents" widget to the Admin Dashboard page, providing administrators with a comprehensive overview of all agents currently running in the system. The changes include both front-end implementation and internationalization support.
- Creates a new AdminDashboardPage with a customizable dashboard layout including active agents and session count widgets
- Refactors existing session components to support scope-based filtering instead of project-specific filtering
- Adds comprehensive internationalization support for "Active Agents" and "Admin Dashboard" across all supported languages
Reviewed Changes
Copilot reviewed 31 out of 31 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| src/components/backend-ai-webui.ts | Adds admin-dashboard to available and superAdminOnly pages |
| react/src/pages/AdminDashboardPage.tsx | New admin dashboard page with Active Agents widget |
| react/src/components/ActiveAgents.tsx | New component displaying active agents in dashboard format |
| react/src/components/AgentList.tsx | Modified to support controlled fetch key and hidden header props |
| react/src/components/MySession.tsx | Refactored to SessionCountDashboardItem with scope-based filtering |
| react/src/components/RecentlyCreatedSession.tsx | Updated to use scope-based filtering instead of project-specific |
| react/src/hooks/useBAISetting.tsx | Added admin_dashboard_board_items setting |
| react/src/App.tsx | Added routing for admin dashboard page |
| react/src/components/MainLayout/WebUISider.tsx | Added admin dashboard menu item |
| resources/i18n/*.json | Added translations for Active Agents and Admin Dashboard |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
f327cee to
24c131f
Compare
24c131f to
988bb6a
Compare
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've left some review regarding the user setting option (toggle visibility for the admin dashboard) and naming conventions. Please check it.
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.
LGTM
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.
please resolve the conflicts
988bb6a to
54a3845
Compare
54a3845 to
22f13d2
Compare
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.
LGTM
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.
LGTM
Merge activity
|
Resolves #4362 ([FR-1531](https://lablup.atlassian.net/browse/FR-1531)) # Add Active Agents Widget to Admin Dashboard This PR adds a new "Active Agents" widget to the Admin Dashboard page, providing an at-a-glance overview of all agents currently running in the system. ## Changes: - Created a new `ActiveAgents` component that displays a compact list of active agents - Added the widget to the Admin Dashboard layout with appropriate sizing - Added translations for "Active Agents" in all supported languages - Exported `BAI_BOARD_ITEM_TITLE_Z_INDEX` constant to ensure consistent z-index values - Modified `AgentList` component to accept controlled fetch key and header props **Checklist:** - [x] Documentation - [ ] Minium required manager version - [ ] Specific setting for review (eg., KB link, endpoint or how to setup) - [ ] Minimum requirements to check during review - [ ] Test case(s) to demonstrate the difference of before/after [FR-1531]: https://lablup.atlassian.net/browse/FR-1531?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
22f13d2 to
7119a62
Compare


Resolves #4362 (FR-1531)
Add Active Agents Widget to Admin Dashboard
This PR adds a new "Active Agents" widget to the Admin Dashboard page, providing an at-a-glance overview of all agents currently running in the system.
Changes:
ActiveAgentscomponent that displays a compact list of active agentsBAI_BOARD_ITEM_TITLE_Z_INDEXconstant to ensure consistent z-index valuesAgentListcomponent to accept controlled fetch key and header propsChecklist: