feat: Add brand manifest policy UI dropdown in Admin #726
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The brand_manifest_policy system (added in PR #663) allows per-tenant configuration of when brand_manifest is required: - require_auth: Auth required, brand_manifest optional (default) - require_brand: Auth + brand_manifest required (strictest) - public: No auth required (no pricing shown) This commit adds: - Documentation explaining the three policy options - Python script to update all tenants to require_auth - Instructions for updating via Admin UI, SQL, or Fly.io To make brand_manifest optional for testing, simply set the tenant's brand_manifest_policy to 'require_auth' instead of 'require_brand'. Related: PR #663
Add UI dropdown in Policies & Workflows section to configure brand_manifest_policy. The brand_manifest_policy system was added in PR #663 with three options: - require_auth (default): Auth required, brand_manifest optional - require_brand: Auth + brand_manifest required (strictest) - public: No auth required, no pricing shown This PR adds the missing UI piece so admins can configure the policy: Changes: - Added dropdown to tenant_settings.html in Policies & Workflows section - Added backend handling in settings.py to save the policy value - Added documentation in docs/brand-manifest-policy.md - Added script to batch update tenants: scripts/update_brand_manifest_policy.py Usage: Admin UI → Tenant Settings → Policies & Workflows → Brand Manifest Policy Related: PR #663 (original implementation)
danf-newton
pushed a commit
to Newton-Research-Inc/salesagent
that referenced
this pull request
Nov 24, 2025
…ol#726) * docs: Add brand manifest policy configuration docs and update script The brand_manifest_policy system (added in PR adcontextprotocol#663) allows per-tenant configuration of when brand_manifest is required: - require_auth: Auth required, brand_manifest optional (default) - require_brand: Auth + brand_manifest required (strictest) - public: No auth required (no pricing shown) This commit adds: - Documentation explaining the three policy options - Python script to update all tenants to require_auth - Instructions for updating via Admin UI, SQL, or Fly.io To make brand_manifest optional for testing, simply set the tenant's brand_manifest_policy to 'require_auth' instead of 'require_brand'. Related: PR adcontextprotocol#663 * feat: Add brand manifest policy UI dropdown in Admin Add UI dropdown in Policies & Workflows section to configure brand_manifest_policy. The brand_manifest_policy system was added in PR adcontextprotocol#663 with three options: - require_auth (default): Auth required, brand_manifest optional - require_brand: Auth + brand_manifest required (strictest) - public: No auth required, no pricing shown This PR adds the missing UI piece so admins can configure the policy: Changes: - Added dropdown to tenant_settings.html in Policies & Workflows section - Added backend handling in settings.py to save the policy value - Added documentation in docs/brand-manifest-policy.md - Added script to batch update tenants: scripts/update_brand_manifest_policy.py Usage: Admin UI → Tenant Settings → Policies & Workflows → Brand Manifest Policy Related: PR adcontextprotocol#663 (original implementation)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds the Admin UI dropdown for configuring the
brand_manifest_policythat was implemented in PR #663.Changes
Admin UI Template (
templates/tenant_settings.html)require_auth(default),require_brand,publicBackend Handler (
src/admin/blueprints/settings.py)tenant.brand_manifest_policyDocumentation (
docs/brand-manifest-policy.md)Update Script (
scripts/update_brand_manifest_policy.py)require_authHow to Use
Via Admin UI:
Policy Options
Testing
This completes the brand_manifest_policy system. The backend was implemented in PR #663, and this PR adds the UI configuration piece.
Related: #663