1392542 - [Leanplum] Add a new trigger for App Start

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Metrics, enhancement)

Description

[Nevin Chen(Not active on Bugzilla)]

Currently we only have event "E_Launch_But_Not_Default_Browser"
We need a normal trigger "E_Launch_Browser"

Comment 1

[Nevin Chen(Not active on Bugzilla)]

Attachment: Bug 1392542 - Add a new trigger for App Start.

Review commit: https://reviewboard.mozilla.org/r/171080/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/171080/

Comment 2

[Nevin Chen(Not active on Bugzilla)]

Attachment: Bug 1392542 - Update document for new trigger - App Start

Review commit: https://reviewboard.mozilla.org/r/171082/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/171082/

Comment 3

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8899744 [details]
Bug 1392542 - Update document for new trigger - App Start

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/171082/diff/1-2/

Comment 4

[Nevin Chen(Not active on Bugzilla)]

We should keep the previous event E_Launch_But_Not_Default_Browser because it's in use already.

Comment 5

[Max Liu [:maliu]]

Comment on attachment 8899743 [details]
Bug 1392542 - Add a new trigger for App Start.

https://reviewboard.mozilla.org/r/171080/#review176588

Comment 6

[Chenxia Liu [:liuche]]

Comment on attachment 8899744 [details]
Bug 1392542 - Update document for new trigger - App Start

https://reviewboard.mozilla.org/r/171082/#review176954

Type 1 and Type 2 data, and it looks like the App Start is documented in the Leanplum Contextual Hints wiki page ( https://wiki.mozilla.org/Leanplum_Contextual_Hints ), so data-review+.

::: mobile/android/docs/mma.rst:151
(Diff revision 2)
> +{
> +  "event" : "E_Launch_But_Not_Default_Browser"
> +}
> +* General app start event
> +{
> +  "event" : "E_Launch_Browser"

Do we get this through coreping or something else? I guess it's hard to use that data across telemetry platforms :/

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bb814f1d5ae0
Add a new trigger for App Start. r=maliu
https://hg.mozilla.org/integration/autoland/rev/764a6904f190
Update document for new trigger - App Start r=liuche

Comment 8

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/bb814f1d5ae0
https://hg.mozilla.org/mozilla-central/rev/764a6904f190