Closed Bug 1860290 Opened 1 year ago Closed 1 year ago

Forgot to include brand.ftl in ExtensionsUI.sys.mjs

Categories

(Thunderbird :: Add-Ons: General, defect)

defect

Tracking

(thunderbird_esr115 fixed)

RESOLVED FIXED
120 Branch
Tracking Status
thunderbird_esr115 --- fixed

People

(Reporter: TbSync, Assigned: TbSync)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

The add-on popups now show untranslated { -brand-short-name }

Assignee: nobody → john
Status: NEW → ASSIGNED

Just to make sure: The [ESR115] marked version of the patch is for 115.4.0. It needs its own patch, because we recently converted ExtensionsUI.jsm to ExtensionsUI.sys.mjs on comm-central.

Comment on attachment 9359570 [details]
Bug 1860290 - [ESR115] Include brand.ftl in ExtensionsUI.jsm for add-on install popups. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): 1842490
User impact if declined: Add-on install popup shows { -brand-short-name }
Testing completed (on c-c, etc.): none
Risk to taking this patch (and alternatives if risky): This patch just adds "branding/brand.ftl" to the Localization object. I could not run eslint, but it should be fine.

Attachment #9359570 - Flags: approval-comm-esr115?

Comment on attachment 9359570 [details]
Bug 1860290 - [ESR115] Include brand.ftl in ExtensionsUI.jsm for add-on install popups. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr115

Attachment #9359570 - Flags: approval-comm-esr115? → approval-comm-esr115+
Target Milestone: --- → 120 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/8374bda2e8fd
Include brand.ftl in ExtensionsUI.sys.mjs for add-on install popups. r=babolivier

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: