-
Notifications
You must be signed in to change notification settings - Fork 420
adding Unified Search & Filter #1541
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
Signed-off-by: NAYANAR <[email protected]>
Signed-off-by: NAYANAR <[email protected]>
Signed-off-by: NAYANAR <[email protected]>
Signed-off-by: NAYANAR <[email protected]>
Signed-off-by: NAYANAR <[email protected]>
rakdutta
left a comment
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.
Unified search works across all Admin UI tabs—MCP Server, Tools, Prompts, Resources, and A2A Agents—but not for Virtual Servers. In the Virtual Servers tab, search is limited to the name field, and across all tabs, the search is also incorrectly matching keywords from the Actions column.
Please update the search logic to use only the intended data fields and exclude values from the Actions column.
Signed-off-by: NAYANAR <[email protected]>
Signed-off-by: NAYANAR <[email protected]>
rakdutta
left a comment
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.
PR is ready to be merged. The unified search and filter functionality is now working across all Admin UI sections — MCP Server, Virtual Server, Tools, Prompts, Resources, and A2A Agent pages.
Signed-off-by: NAYANAR [email protected]
💡 Fix Description
Implement unified search and filter functionality across all Admin UI tabs Servers, Virtual Servers, Tools, Prompts, Resources, A2A Agents
🧪 Verification
make lintmake test📐 MCP Compliance (if relevant)
✅ Checklist
make black isort pre-commit)