1862678 - Add telemetry for mail.delete_matches_sort_order

Status: RESOLVED FIXED

(Thunderbird :: Folder and Message Lists, task, P1)

Description

[Martin Giger [:freaktechnik]]

We want to add telemetry for the mail.delete_matches_sort_order boolean pref to help decide if we need to keep supporting it.

The first step is to submit a data collection request. See https://wiki.mozilla.org/Thunderbird/Data_Collection for instructions.

The actual telemetry collection will happen in MailGlue.jsm, where you can add the pref to the array of boolean prefs we collect via telemetry: https://searchfox.org/comm-central/rev/f7ee90e754987bb5d32ca07311eca80e4f703f49/mail/components/MailGlue.jsm#1165

We will want to uplift this to 115.

Comment 1

[solange]

Attachment: 1862678_DataCollectionRequest.md

Comment 2

[Andrei Hajdukewycz [:sancus]]

Comment on attachment 9366966 [details]
1862678_DataCollectionRequest.md

This is approved as another boolean referencing Bug 1757993's approval for all booleans.

Comment 3

[solange]

Attachment: Bug 1862678 - Add telemetry for mail.delete_matches_sort_order. r=#thunderbird-reviewers

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/cb473f5d661b
Add telemetry for mail.delete_matches_sort_order. r=leftmostcat

Comment 5

[Wayne Mery (:wsmwk)]

sol,
I don't feel this needs to go through beta before uplift to 115. We are building 115 this week.
If you agree, please request uplift.

Comment 6

[solange]

Comment on attachment 9366969 [details]
Bug 1862678 - Add telemetry for mail.delete_matches_sort_order. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined: lack of telemetry would hinder our decision to mantain the preference
Testing completed (on c-c, etc.): -
Risk to taking this patch (and alternatives if risky): none

[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined: lack of telemetry would hinder our decision to mantain the preference
Testing completed (on c-c, etc.): -
Risk to taking this patch (and alternatives if risky): none

Comment 7

[Wayne Mery (:wsmwk)]

Comment on attachment 9366969 [details]
Bug 1862678 - Add telemetry for mail.delete_matches_sort_order. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr115

There are no more betas in this cycle, so the patch will be in 122 beta after Monday's merge.

Comment 8

[Rob Lemley [:rjl]]

Thunderbird 115.6.0:
https://hg.mozilla.org/releases/comm-esr115/rev/8c3342166ba4