Skip to content

Remove companyURL and align branding docs/config with explicit Support/Privacy/About URLs#20

Draft
Copilot wants to merge 14 commits intodevfrom
copilot/fix-support-privacy-buttons
Draft

Remove companyURL and align branding docs/config with explicit Support/Privacy/About URLs#20
Copilot wants to merge 14 commits intodevfrom
copilot/fix-support-privacy-buttons

Conversation

Copy link

Copilot AI commented Feb 28, 2026

companyURL had become redundant after introducing explicit branding links for support, privacy, and about. This PR removes companyURL across runtime config surfaces and updates branding documentation/examples to reflect the current model.

  • Runtime branding model cleanup

    • Removed companyURL from branding defaults and managed schema:
      • config/branding.json
      • config/managed_schema.json
      • scripts/modules/config-manager.js
    • Removed legacy privacy fallback derived from companyURL; privacy now comes from explicit privacyPolicyUrl or default branding config.
  • Options + popup behavior

    • Removed Company URL input and persistence wiring from options:
      • options/options.html
      • options/options.js
    • Removed popup company-link handling (companyLink + "company" footer route); company branding remains visible as text:
      • popup/popup.html
      • popup/popup.js
  • Enterprise policy surface updates

    • Removed companyURL from policy artifacts/examples (Windows scripts/registry, ADMX/ADML, Firefox/macOS/Linux policy JSON):
      • enterprise/Deploy-Windows-Chrome-and-Edge.ps1
      • enterprise/Remove-Windows-Chrome-and-Edge.ps1
      • enterprise/Check-Extension-Policy.reg
      • enterprise/admx/Check-Extension.admx
      • enterprise/admx/en-US/Check-Extension.adml
      • enterprise/firefox/policies.json
      • enterprise/macos-linux/chrome-managed-policy.json
      • enterprise/macos-linux/edge-managed-policy.json
      • enterprise/Test-Extension-Policy.ps1
  • Documentation alignment

    • Updated branding docs to document current link fields (supportUrl, privacyPolicyUrl, aboutUrl) and kept naming consistent:
      • docs/settings/branding.md
  • Focused test expectation update

    • Updated config persistence test to remove companyURL usage and assert fallback URLs from default branding config:
      • tests/config-persistence.test.js
// config-manager.js
if (!finalBranding.supportUrl && finalBranding.supportEmail) {
  finalBranding.supportUrl = `mailto:${finalBranding.supportEmail}`;
}
// removed: privacy fallback from companyURL

https://github.com/user-attachments/assets/7aa7e466-66fb-4eee-8a85-0799431940e6

Original prompt

Branding options manually in the GUI and the Support and Privacy buttons still don't do anything. The About button works fine. Please check they are wired correctly and also make sure they work in the enterprise deployment method as well


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 28, 2026 03:42
Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix wired issues with Support and Privacy buttons Fix popup Support/Privacy links for manual and enterprise branding configurations Feb 28, 2026
…ise policy

Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Copilot AI changed the title Fix popup Support/Privacy links for manual and enterprise branding configurations Expose Support/Privacy branding URLs in UI and enterprise policy, and wire company URL consistently Feb 28, 2026
…cies

Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Copilot AI changed the title Expose Support/Privacy branding URLs in UI and enterprise policy, and wire company URL consistently Add configurable About URL across branding UX and enterprise policy (with popup fallback behavior) Feb 28, 2026
Copilot AI and others added 4 commits February 28, 2026 05:47
Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Copilot AI changed the title Add configurable About URL across branding UX and enterprise policy (with popup fallback behavior) Remove obsolete companyURL branding field across UI, config, and enterprise policy surfaces Feb 28, 2026
Copilot AI and others added 3 commits February 28, 2026 05:59
Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Co-authored-by: Zacgoose <107489668+Zacgoose@users.noreply.github.com>
Copilot AI changed the title Remove obsolete companyURL branding field across UI, config, and enterprise policy surfaces Remove companyURL and align branding docs/config with explicit Support/Privacy/About URLs Feb 28, 2026
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