[Leanplum] Add a new trigger for App Start

RESOLVED FIXED in Firefox 57

Status

()

Firefox for Android
Metrics
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: nechen, Assigned: nechen)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

(Whiteboard: [FNC][SPT57.2][INT][LP_M2][SP=X, 1])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Assignee)

Description

3 months ago
Currently we only have event "E_Launch_But_Not_Default_Browser"
We need a normal trigger "E_Launch_Browser"
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 4

3 months ago
We should keep the previous event E_Launch_But_Not_Default_Browser because it's in use already.
Flags: needinfo?(liuche)
(Assignee)

Updated

3 months ago
Assignee: nobody → cnevinchen
Whiteboard: [FNC][SPT#57.2][INT]

Comment 5

3 months ago
mozreview-review
Comment on attachment 8899743 [details]
Bug 1392542 - Add a new trigger for App Start.

https://reviewboard.mozilla.org/r/171080/#review176588
Attachment #8899743 - Flags: review?(max) → review+

Comment 6

3 months ago
mozreview-review
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 :/
Attachment #8899744 - Flags: review?(liuche) → review+
Flags: needinfo?(liuche)

Comment 7

3 months ago
Pushed by nechen@mozilla.com:
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
https://hg.mozilla.org/mozilla-central/rev/bb814f1d5ae0
https://hg.mozilla.org/mozilla-central/rev/764a6904f190
Status: NEW → RESOLVED
Last Resolved: 3 months ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Whiteboard: [FNC][SPT#57.2][INT] → [FNC][SPT57.2][INT]
Whiteboard: [FNC][SPT57.2][INT] → [FNC][SPT57.2][INT][LP_M2]
Blocks: 1351571
(Assignee)

Updated

3 months ago
Whiteboard: [FNC][SPT57.2][INT][LP_M2] → [FNC][SPT57.2][INT][LP_M2][SP=X, 1]
You need to log in before you can comment on or make changes to this bug.