1194896 - Add telemetry notification for interacting with data policy notification

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Chenxia Liu [:liuche]]

Add telemetry for launching settings for data notification to the Android notification bar - do people interact with this notification?

Comment 1

[Chenxia Liu [:liuche]]

Attachment: MozReview Request: Bug 1194896 - Add telemetry notification for interacting with data policy notification. r=mcomella

Bug 1194896 - Add telemetry notification for interacting with data policy notification. r=mcomella

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

https://reviewboard.mozilla.org/r/16181/#review14473

::: mobile/android/base/preferences/GeckoPreferences.java:421
(Diff revision 1)
> +            Telemetry.sendUIEvent(TelemetryContract.Event.LAUNCH, Method.NOTIFICATION, "settings-data-choices");

Might be nice to have two probes: One for showing the notification and one for launching from it. That way we can get an estimate of people ignoring it.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to Mark Finkle (:mfinkle) from comment #2)
> https://reviewboard.mozilla.org/r/16181/#review14473
> 
> ::: mobile/android/base/preferences/GeckoPreferences.java:421
> (Diff revision 1)
> > +            Telemetry.sendUIEvent(TelemetryContract.Event.LAUNCH, Method.NOTIFICATION, "settings-data-choices");
> 
> Might be nice to have two probes: One for showing the notification and one
> for launching from it. That way we can get an estimate of people ignoring it.

Let me step back from asking for more probes, and instead just say: If the existing probe in the patch is enough to answer the desired question, then that's all we should add.

Comment 4

[Chenxia Liu [:liuche]]

We actually already have a probe for showing the notification, but we never added one for launching from it :P

Comment 5

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Comment on attachment 8648295 [details]
MozReview Request: Bug 1194896 - Add telemetry notification for interacting with data policy notification. r=mcomella

https://reviewboard.mozilla.org/r/16181/#review14599

Ship It!

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/8090c77871a3

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/8090c77871a3