Closed Bug 1137259 Opened 5 years ago Closed 5 years ago

Don't send Campaign:Set for downloadable distributions

Categories

(Firefox for Android :: General, defect)

37 Branch
All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 39
Tracking Status
firefox37 --- fixed
firefox38 --- fixed
firefox39 --- fixed
fennec 37+ ---

People

(Reporter: rnewman, Assigned: rnewman)

Details

Attachments

(1 file)

07:40:12 <@mfinkle> i still think our java code should do better at splitting the distro and campaign
07:40:58 <@mfinkle> imo, propagateMozillaCampaign(referrer) should not be called for a distro

...

07:42:39 <@mfinkle> Campaign:Set creates a JSON file that will try to update the prefs
07:42:52 <@mfinkle> and create a race with the preferences.json shipped with the distro
07:43:14 <@mfinkle> that is the ONLY purpose of propagateMozillaCampaign

...

07:45:04 < rnewman> that's what the campaign message thing is really doing: turning campaigns into distributions so they go via BLP
07:45:30 <@mfinkle> "turning campaigns into distributions so they go via BLP" - right
07:45:47 <@mfinkle> but i don't think we should turn failed distros back into campaigns
Comment on attachment 8569934 [details] [diff] [review]
Don't send Campaign:Set for distribution referrer intents. v1

Let's update:
https://wiki.mozilla.org/Mobile/Distribution_Files#Crafting_a_distribution_referrer

This patch makes utm_source=mozilla mandatory for campaigns and distributions
Attachment #8569934 - Flags: review?(mark.finkle) → review+
ni for uplift.
Flags: needinfo?(rnewman)
Karen: the result of this, when it reaches release, is that the experiment you were just trying (using a distribution-esque URL instead of a normal campaign URL) won't be recorded as playstore/distribution.
https://hg.mozilla.org/mozilla-central/rev/d40e3d27f8e1
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
Comment on attachment 8569934 [details] [diff] [review]
Don't send Campaign:Set for distribution referrer intents. v1

This patch changes how Play Store builds handle the referrer intent we're sent after install.

The result of the change is that we won't set playstore/distribution for distribution builds that fail to download, or where the distribution isn't found.

[Feature/regressing bug #]:
  N/A

[User impact if declined]:
  N/A

[Describe test coverage new/current, TreeHerder]:
  Distributions have automated tests. Otherwise, effectively untested.

[Risks and why]: 
  Low risk. Actually removes a race condition, so generally positive.

[String/UUID change made/needed]:
  None.
Flags: needinfo?(rnewman)
Attachment #8569934 - Flags: approval-mozilla-beta?
Attachment #8569934 - Flags: approval-mozilla-aurora?
Comment on attachment 8569934 [details] [diff] [review]
Don't send Campaign:Set for distribution referrer intents. v1

Early in beta still so OK to uplift this and evaluate with the larger population.
Attachment #8569934 - Flags: approval-mozilla-beta?
Attachment #8569934 - Flags: approval-mozilla-beta+
Attachment #8569934 - Flags: approval-mozilla-aurora?
Attachment #8569934 - Flags: approval-mozilla-aurora+
tracking-fennec: ? → 37+
You need to log in before you can comment on or make changes to this bug.