2010597 - Update widget telemetry to pass widget name as a param

Status: RESOLVED FIXED

(Firefox :: New Tab Page, enhancement)

Description

[Maxx Crawford [:maxx]]

Summary: Create a few broad widget telemetry events (that include a widget_name param), so we can launch multiple widgets without the need for bespoke telemetry updates for each.

Note: We want to send duplicate events until DS can confirm the new probes work as expected.

Tasks:

• Define new entries in metrics.yaml
• Add new Glean event calls in existing code (TelemetryFeed.sys.mjs example)
• (Follow-up, separate patch) Remove previous events

Comment 1

[Maxx Crawford [:maxx]]

(Duplicated comment from the summary. Please ignore!)

Comment 2

[Maxx Crawford [:maxx]]

Attachment: Bug 2010597 - Add unified widget telemetry metrics. r?#home-newtab-reviewers

Adds three new unified Glean metrics to replace per-widget telemetry:

• widgets_impression: Records when any widget is visible
• widgets_user_event: Records user interactions with any widget
• widgets_enabled: Records when widgets are enabled/disabled

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/a7506b3f4c06
https://hg.mozilla.org/integration/autoland/rev/8ac74d254a74
Add unified widget telemetry metrics. r=home-newtab-reviewers,reemhamz,nina-py

Comment 4

[Cristina Horotan [:chorotan]]

Backed out for causing newtab failure and bc failure

Backout link

Push with failures

Failure log
Failure log bc

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/56c551217d51
https://hg.mozilla.org/integration/autoland/rev/5d4a7cef7eb5
Add unified widget telemetry metrics. r=home-newtab-reviewers,reemhamz,nina-py

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/5d4a7cef7eb5