1862676 - Add telemetry for mailnews.scroll_to_new_message

Status: RESOLVED FIXED

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

Description

[Martin Giger [:freaktechnik]]

We want to add telemetry for the mailnews.scroll_to_new_message 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

[Vineet Deo]

Attachment: 1862676_DataCollectionRequest.txt

Comment 2

[Vineet Deo]

Attachment: Bug 1862676 - Add telemetry for mail.mailnews.scroll_to_new_message pref. r=#thunderbird-reviewers

Comment 3

[Andrei Hajdukewycz [:sancus]]

Comment on attachment 9363409 [details]
1862676_DataCollectionRequest.txt

This is just an additional boolean pref and is fine under the same approval as the others from Bug 1757993.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/00f2ca50b046
Add telemetry for mail.mailnews.scroll_to_new_message pref. r=mkmelin

Comment 5

[Vineet Deo]

Comment on attachment 9363413 [details]
Bug 1862676 - Add telemetry for mail.mailnews.scroll_to_new_message pref. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined: None.
Testing completed (on c-c, etc.): -
Risk to taking this patch (and alternatives if risky): None, this is a telemetry bug.

Comment 6

[Wayne Mery (:wsmwk)]

Comment on attachment 9363413 [details]
Bug 1862676 - Add telemetry for mail.mailnews.scroll_to_new_message pref. r=#thunderbird-reviewers

[Triage Comment]
Approved for beta

Comment 7

[Daniel Darnell [:dandarnell]]

Thunderbird 121.0b6:
https://hg.mozilla.org/releases/comm-beta/rev/ed2fc240f42a

Comment 8

[Wayne Mery (:wsmwk)]

Vineet, we are building 115 this week.
It's a safe patch, so if you are ready please request uplift.

Comment 9

[Vineet Deo]

Comment on attachment 9363413 [details]
Bug 1862676 - Add telemetry for mail.mailnews.scroll_to_new_message pref. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined: None.
Testing completed (on c-c, etc.): -
Risk to taking this patch (and alternatives if risky): None. This is a telemetry bug.

Comment 10

[Wayne Mery (:wsmwk)]

Comment on attachment 9363413 [details]
Bug 1862676 - Add telemetry for mail.mailnews.scroll_to_new_message pref. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr115

Comment 11

[Rob Lemley [:rjl]]

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