Closed Bug 1373818 Opened 5 years ago Closed 5 years ago

Treat Adjust distribution parameter the same as the mozilla parameter


(Firefox for Android Graveyard :: General, enhancement)

Not set


(firefox55 fixed, firefox56 fixed)

Firefox 56
Tracking Status
firefox55 --- fixed
firefox56 --- fixed


(Reporter: mkaply, Assigned: mkaply)




(1 file)

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:


and adjust will be able to track that even though it didn't originate with an Adjust URL.
Comment on attachment 8878673 [details]
Bug 1373818 - Don't hardcode our tracker per Adjust.

I believe this does what you intend it to.
Attachment #8878673 - Flags: review?(rnewman) → review+
hg error in cmd: hg push -r tip ssh:// pushing to ssh://
searching for changes
remote: adding changesets
remote: adding manifests
remote: adding file changes
remote: added 1 changesets with 1 changes to 1 files
remote: ************************** ERROR ****************************
remote: Rev d6e88a4c9667 needs "Bug N" or "No bug" in the commit message.
remote: Michael Kaply <>
remote: Bug #1373818 - Allow Adjust campaign ID to behave as OTA distribution. r=rnewman
remote: MozReview-Commit-ID: 8MEQ84PuH7i
remote: *************************************************************
remote: transaction abort!
remote: rollback completed
remote: pretxnchangegroup.c_commitmessage hook failed
abort: push failed on remote
Pushed by
Allow Adjust campaign ID to behave as OTA distribution. r=rnewman
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
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+
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.
Adjust changed the API on us, so sadly new patch coming.
Resolution: FIXED → ---

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)
Will do.
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
See Also: → 1382677
Pushed by
Don't hardcode our tracker per Adjust. r=rnewman
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.