Skip to content

Bug 2012458 - Add policy for configuring sidebar AI providers.#473

Open
mkaply wants to merge 3 commits intomozilla:enterprise-mainfrom
mkaply:bug2012458_chatsidebar_policy
Open

Bug 2012458 - Add policy for configuring sidebar AI providers.#473
mkaply wants to merge 3 commits intomozilla:enterprise-mainfrom
mkaply:bug2012458_chatsidebar_policy

Conversation

@mkaply
Copy link
Contributor

@mkaply mkaply commented Feb 23, 2026

This also has a change to allow external icons to work in the sidebar.

I focused on adding/removing providers and removing prompts.

If we need to add prompts, it would be straightforward, but that wasn't part of the original requirements.

@mkaply mkaply requested review from a team and 1rneh February 23, 2026 21:28
Copy link
Contributor

@1rneh 1rneh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the patch! This is my first round of review. I wasn’t previously familiar with the GenAI system module or the sidebar integration, so these comments are based on my current understanding.

I’ve included some questions and a few concerns regarding how prompts and providers are currently being cleared.

@mkaply
Copy link
Contributor Author

mkaply commented Mar 3, 2026

I have a couple more things to fix, but this is basically a rewrite that makes things much simpler and will make things much easier to undo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants