Treat Adjust distribution parameter the same as the mozilla parameter

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: mkaply, Assigned: mkaply)

Tracking

unspecified
Firefox 56
Points:
---

Firefox Tracking Flags

(firefox55 fixed, firefox56 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
So we're working with Adjust to try to track installs that happen with a referrer intent.

To do that, they are enabling using the utm_source parameter to pass an Adjust tracking token.

For our purposes, that means we need to treat the adjust tracking token that we've already assigned for distributions as if it is the utm_source "mozilla"

The token in question is xodfft.

When third parties use a referrer intent, they will send it to us like this:

utm_source=xodfft&utm_content=wildtangent&utm_campaign=distribution&utm_term=browse&utm_store=true"

and adjust will be able to track that even though it didn't originate with an Adjust URL.
Comment hidden (mozreview-request)

Comment 2

2 years ago
mozreview-review
Comment on attachment 8878673 [details]
Bug 1373818 - Don't hardcode our tracker per Adjust.

https://reviewboard.mozilla.org/r/149978/#review154674

I believe this does what you intend it to.
Attachment #8878673 - Flags: review?(rnewman) → review+

Comment 3

2 years ago
hg error in cmd: hg push -r tip ssh://hg.mozilla.org/integration/autoland: pushing to ssh://hg.mozilla.org/integration/autoland
searching for changes
remote: adding changesets
remote: adding manifests
remote: adding file changes
remote: added 1 changesets with 1 changes to 1 files
remote: 
remote: 
remote: ************************** ERROR ****************************
remote: Rev d6e88a4c9667 needs "Bug N" or "No bug" in the commit message.
remote: Michael Kaply <mozilla@kaply.com>
remote: Bug #1373818 - Allow Adjust campaign ID to behave as OTA distribution. r=rnewman
remote: 
remote: MozReview-Commit-ID: 8MEQ84PuH7i
remote: *************************************************************
remote: 
remote: 
remote: transaction abort!
remote: rollback completed
remote: pretxnchangegroup.c_commitmessage hook failed
abort: push failed on remote
Comment hidden (mozreview-request)

Comment 5

2 years ago
Pushed by mozilla@kaply.com:
https://hg.mozilla.org/integration/autoland/rev/12f0f85e4ade
Allow Adjust campaign ID to behave as OTA distribution. r=rnewman
https://hg.mozilla.org/mozilla-central/rev/12f0f85e4ade
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
(Assignee)

Comment 7

2 years ago
Comment on attachment 8878673 [details]
Bug 1373818 - Don't hardcode our tracker per Adjust.

Approval Request Comment
[Feature/Bug causing the regression]: Allow Adjust tracking in more situations.
[User impact if declined]: No user impact. Allow us better tracking.
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes.
[Needs manual test from QE? If yes, steps to reproduce]: No.
[List of other uplifts needed for the feature/fix]: None.
[Is the change risky?]: Extremely low.
[Why is the change risky/not risky?]: Only used when a referrer intent is executed
[String changes made/needed]: None

This patch basically gives us another parameter that marks something as an OTA distribution but in this case, it also sends additional info to Adjust to allow it to be tracked.

Adjust is targeting an update fairly soon to enable this functionality, so we want to get it in as soon as possible.
Attachment #8878673 - Flags: approval-mozilla-beta?
Comment on attachment 8878673 [details]
Bug 1373818 - Don't hardcode our tracker per Adjust.

install tracking for fennec, beta55+
Attachment #8878673 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(Assignee)

Comment 9

2 years ago
Adjust changed the API on us so we don't need this in beta. There will be a different patch. Sorry for the duplicate work here.
(Assignee)

Comment 11

2 years ago
Adjust changed the API on us, so sadly new patch coming.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment hidden (mozreview-request)
(Assignee)

Comment 13

a year ago
Ryan:

I ended up needing to change the name of the tracker, but it's the same review request, so I don't see how to ask for this update to go to beta as well. Thanks for any help you can provide.
Flags: needinfo?(ryanvm)
From a work tracking standpoint, it would probably be better to just file a new bug for renaming it.
Flags: needinfo?(ryanvm)
(Assignee)

Comment 15

a year ago
Will do.
Status: REOPENED → RESOLVED
Last Resolved: 2 years agoa year ago
Resolution: --- → FIXED
(Assignee)

Updated

a year ago
See Also: → bug 1382677

Comment 16

a year ago
Pushed by mozilla@kaply.com:
https://hg.mozilla.org/integration/autoland/rev/77a37c261feb
Don't hardcode our tracker per Adjust. r=rnewman
You need to log in before you can comment on or make changes to this bug.