feat(intelligent-assistant): implement permission-denied UI for chat, notebooks, and MCP tools - #4604
Conversation
… notebooks, and MCP tools Fixes: https://redhat.atlassian.net/browse/RHIDP-15932 Fixes: https://redhat.atlassian.net/browse/RHIDP-15933 Fixes: https://redhat.atlassian.net/browse/RHIDP-15934 Signed-off-by: rohitratannagar <rohitratannagar2003@gmail.com> Co-authored-by: Cursor <cursoragent@cursor.com>
|
Important This PR includes changes that affect public-facing API. Please ensure you are adding/updating documentation for new features or behavior. Changed Packages
|
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #4604 +/- ##
=======================================
Coverage 62.38% 62.39%
=======================================
Files 2607 2607
Lines 104787 104807 +20
Branches 29432 29466 +34
=======================================
+ Hits 65375 65394 +19
+ Misses 38856 38854 -2
- Partials 556 559 +3
*This pull request uses carry forward flags. Click here to find out more. Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
michael-valdron
left a comment
michael-valdron
left a comment
There was a problem hiding this comment.
Hi @rohitratannagar,
Thanks for taking care of the frontend work and getting started.
Before we proceed I want to inform you of some major revisions pushed by UI team and the PM, see: https://redhat.atlassian.net/browse/RHIDP-16542
I'll be working on library/backend changes starting today and should take me a day or two to complete. Once its done, these changes can be adjusted to the new revised changes.
michael-valdron
left a comment
michael-valdron
left a comment
There was a problem hiding this comment.
#4612 should merge before this one.
michael-valdron
left a comment
michael-valdron
left a comment
There was a problem hiding this comment.
#4612 is merged ✔️
@rohitratannagar You can now proceed to adjust the changes here for the #4612 revised permission sets.
2 participants
Description
Implements permission-denied UI screens for three permission sets in the Intelligent Assistant plugin:
RHIDP-15932 — Chat permissions (
chat.use)intelligent-assistant.chat.useis denied, the MessageBar is fully disabled (no typing, no send button, no attach/mic buttons)RHIDP-15933 — Notebook permissions (
notebooks.use/notebooks.manage)intelligent-assistant.notebooks.useis denied, the entire Notebooks tab disappearsintelligent-assistant.notebooks.manageis denied, rename and delete actions are disabled with explanatory tooltipsRHIDP-15934 — MCP tools permissions (
mcp.tools.use)mcp.tools.useis denied, the MCP Settings menu item is disabled with a tooltipmcp.tools.manageread-only behavior is preservedTranslation updates
All permission strings are translated in all 6 supported locales (en, de, es, fr, it, ja).
Fixed
Checklist
npx tsc --noEmit)Made with Cursor