Closed
Bug 1310146
Opened 8 years ago
Closed 8 years ago
Add sync client counts to UITour getConfiguration
Categories
(Firefox :: Tours, defect)
Firefox
Tours
Tracking
()
RESOLVED
FIXED
Firefox 52
People
(Reporter: Gijs, Assigned: osmose)
References
Details
Attachments
(1 file)
40 bytes,
text/x-review-board-request
|
ritu
:
approval-mozilla-aurora+
ritu
:
approval-mozilla-beta+
|
Details |
Patches were written and reviewed + data-reviewed in bug 1305878. Split off to keep uplifts sane.
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/cf3c0bd40b8f Add sync client counts to UITour, r=gijs,data-r=bsmedberg
Comment 2•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/cf3c0bd40b8f
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Assignee | ||
Comment 3•8 years ago
|
||
Approval Request Comment [Feature/regressing bug #]: 1305878 [User impact if declined]: Product wants this data available via UITour for targeted messaging promoting Firefox for Android to users who have sync on desktop but not mobil. Without this patch, they can't target that messaging properly. [Describe test coverage new/current, TreeHerder]: Added mochitests for the additions to the UITour API. [Risks and why]: This is pretty low-risk; only a small subset of websites (including SHIELD/Normandy, the primary users of this patch) even have access to UITour. The change itself is small and uses safe APIs, and the failure case is these sites not being able to access this data we want. The upside is that we can much more effectively target the mobile promotion (and other SHIELD recipes) and avoid having to do widespread mistargeted promos to get the engagement we want. [String/UUID change made/needed]: None
Attachment #8802290 -
Flags: approval-mozilla-beta?
Attachment #8802290 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 4•8 years ago
|
||
Oh yeah, I forgot to mention that the request for uplift is tied to the timing of the mobile promo, which is aiming to happen before the end of the year. Hence why we want to get into 50.
Updated•8 years ago
|
status-firefox50:
--- → affected
status-firefox51:
--- → affected
Comment on attachment 8802290 [details] Bug 1305878 - Add sync client counts to UITour. This is needed for filtering through a huge set of end-users down to a selective set for targeted experiments, fix has stabilized on Nightly52, Aurora51+, Beta50+
Attachment #8802290 -
Flags: approval-mozilla-beta?
Attachment #8802290 -
Flags: approval-mozilla-beta+
Attachment #8802290 -
Flags: approval-mozilla-aurora?
Attachment #8802290 -
Flags: approval-mozilla-aurora+
Hi Mike, have we verified on Nightly52, that Shield has access to the data via these new UITour APIs or not? Just want to make sure this is well tested and there won't any last minute uplifts in Beta50.
Flags: needinfo?(mkelly)
Assignee | ||
Comment 7•8 years ago
|
||
The patch to SHIELD to read these values hasn't been written yet, so I haven't tested it directly yet. But, testing on another UITour-whitelisted site has proven in the past to be a good proxy for testing UITour, and I've tested on the latest nightly by going to support.mozilla.org and calling the API from the web console:
> >> Mozilla.UITour.getConfiguration('sync', console.log.bind(console))
> Object { setup: true, desktopDevices: 2, mobileDevices: 1, 1 more… }
My plan is to work on the patch for SHIELD tomorrow; once that's done, I can spoof the self-repair domain and test locally to see if it'll work. I'm pretty happy with the test above, though. I'll update this bug in either case with the results of my test.
Flags: needinfo?(mkelly)
Comment 8•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/9995bef459e9
Comment 9•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/c5487cf50dbc
Assignee | ||
Comment 10•8 years ago
|
||
SHIELD patch is ready at https://github.com/mozilla/normandy/pull/302. Using that patch I tested against Nightly and was able to target a recipe successfully based on the number of sync clients I had configured.
You need to log in
before you can comment on or make changes to this bug.
Description
•