Closed Bug 1842490 Opened 3 years ago Closed 3 years ago

Missing text / strings for permissions in addons manager

Categories

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

Thunderbird 115
defect

Tracking

(thunderbird_esr115+ fixed, thunderbird119 fixed)

RESOLVED FIXED
120 Branch
Tracking Status
thunderbird_esr115 + fixed
thunderbird119 --- fixed

People

(Reporter: Thunderbird_Mail_DE, Assigned: john)

References

Details

Attachments

(1 file, 1 external file)

Attached image Screenshot —

In Thunderbird 115 RC build 1 the strings for permissions in addons manager are missing and you can only read "null" for these WebExtensions permissions. This is for different locales like Dutch and German - maybe all locales?

This is partially being taken care of in https://bugzilla.mozilla.org/show_bug.cgi?id=1852249. Once that landed, we can fix Thunderbird.

Depends on: 1852249
Assignee: nobody → john
Status: NEW → ASSIGNED
Blocks: 1854074
Target Milestone: --- → 120 Branch

Sorry this has some (non-trivial) linting issues.

I think Martin actually pointed that out to me earlier and I forgot about it. You will get a review shortly.

Pushed by martin@humanoids.be:
https://hg.mozilla.org/comm-central/rev/0d6b81acbc27
Fix missing descriptions for permissions in addons manager. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

Comment on attachment 9354056 [details]
Bug 1842490 - Fix missing descriptions for permissions in addons manager. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): 1793557
User impact if declined: Missing permission descriptions in the permission tab of AOM
Testing completed (on c-c, etc.): 2 days
Risk to taking this patch (and alternatives if risky): low (tested locally, covered by m-c tests)

Attachment #9354056 - Flags: approval-comm-beta?

Comment on attachment 9354056 [details]
Bug 1842490 - Fix missing descriptions for permissions in addons manager. r=#thunderbird-reviewers

[Triage Comment]
Approved for beta

Attachment #9354056 - Flags: approval-comm-beta? → approval-comm-beta+

I manually tested https://treeherder.mozilla.org/jobs?repo=comm-beta&revision=de771cb0153e527e1251d902d644ee79c3596576&selectedTaskRun=ZxXTH71lTkGb_lgnx3bhPQ.0 and it is fixed.

The corresponding m-c patch has been uplifted to m-c ESR115, so we can uplift this to c-c ESR115 now as well.

Comment on attachment 9354056 [details]
Bug 1842490 - Fix missing descriptions for permissions in addons manager. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): 1793557
User impact if declined: Missing permission descriptions in the permission tab of AOM
Testing completed (on c-c, etc.): successfully tested on beta (https://treeherder.mozilla.org/jobs?repo=comm-beta&revision=de771cb0153e527e1251d902d644ee79c3596576&selectedTaskRun=ZxXTH71lTkGb_lgnx3bhPQ.0)
Risk to taking this patch (and alternatives if risky): low (tested locally, covered by m-c tests)

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

Comment on attachment 9354056 [details]
Bug 1842490 - Fix missing descriptions for permissions in addons manager. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr115

Attachment #9354056 - Flags: approval-comm-esr115? → approval-comm-esr115+

Backout comm-esr115:
https://hg.mozilla.org/releases/comm-esr115/rev/d78c0c2372da

The GRE bug is not in 115.3.x, it won't be until 115.4.0.
All tests are failing miserably because of this.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 3 years ago → 3 years ago
Resolution: --- → FIXED
Whiteboard: [TM 115.4.0]
Whiteboard: [TM 115.4.0]
Regressions: 1860290
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: