Closed
Bug 1216225
Opened 9 years ago
Closed 9 years ago
Add function to provide accurate Firefox channel/version info based on mozUITour API
Categories
(www.mozilla.org :: Pages & Content, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kohei, Assigned: kohei)
References
Details
(Whiteboard: [kb=1876707])
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
Details |
My pull request for Bug 939470 is now ready, but before solving the bug on the critical /firefox/new/ page, we need some tests on other pages using GA events so we can analyze how many people are hitting the fallback code and how many people are using a minor Firefox release, etc.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Comment 2•9 years ago
|
||
Hi Gareth, I'm adding a new GA event here. It happens when the user's Firefox version number and channel info is retrieved via the mozUITour API, and includes the version, channel, whether the version is up-to-date or not, and whether the info is accurate or not (in case the API didn't work for some reason). The event will be removed maybe in a month once we gain an insight into the accuracy of the API and the freshness of Firefox. Does it work for you?
https://github.com/kyoshino/bedrock/commit/995d7b008acf5057c7ccce287a15760df6fba952#diff-8e44ab1d2ba495153c12b57837b6d8eeR134
Flags: needinfo?(garethcull.bugs)
Comment 3•9 years ago
|
||
This is now up on demo2:
Pages that will need testing:
https://www-demo2.allizom.org/en-US/plugincheck/
https://www-demo2.allizom.org/en-US/firefox/desktop/
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/
https://www-demo2.allizom.org/en-US/firefox/products/
Note: we're not adding this to /firefox/new/ until after we're confident the data looks OK.
Comment 4•9 years ago
|
||
Thanks guys.
Alex,
Can I snag 10 minutes of your time Alex early next Tuesday or Wed to review? I'll ping you in IRC and we can hop on vidyo if that works. Thanks.
Gareth
Flags: needinfo?(garethcull.bugs)
Comment 5•9 years ago
|
||
(In reply to Gareth Cull [:garethc] from comment #4)
> Thanks guys.
>
> Alex,
>
> Can I snag 10 minutes of your time Alex early next Tuesday or Wed to review?
> I'll ping you in IRC and we can hop on vidyo if that works. Thanks.
>
> Gareth
Sure, happy to talk you through what we're doing and what we'd like to get some data on. Thanks, Gareth!
Comment 6•9 years ago
|
||
Testing notes for Firefox 42 (release)
--------------------------------------
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = true
Mozilla.Client.FirefoxDetails = { version: "42.0", channel: "release", isUpToDate: true, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - no download button visible (PASS)
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = false
Mozilla.Client.FirefoxDetails = { version: "42.0", channel: "release", isUpToDate: true, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - no download button visible (PASS)
The only small issue I noticed was on the the /firefox/products/ page, where there is now a visible flash of the download prompt before it disappears again. This is visible with uitour both enabled and disabled.
Comment 7•9 years ago
|
||
Testing notes for Firefox 38.4.0 (ESR)
--------------------------------------
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = true
Mozilla.Client.FirefoxDetails = { accurate: true, version: "38.4.0", channel: "esr", isUpToDate: true, isESR: true }
https://www-demo2.allizom.org/en-US/plugincheck/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - no download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - no download button visible (PASS)
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = false
Mozilla.Client.FirefoxDetails = { accurate: false, version: "38.0", channel: "release", isUpToDate: false, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - download button visible (PASS)
Comment 8•9 years ago
|
||
Testing notes for Firefox 38.0.6 (Release)
------------------------------------------
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = true
Mozilla.Client.FirefoxDetails = { accurate: true, version: "38.0.6", channel: "release", isUpToDate: false, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - download button visible (PASS)
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = false
Mozilla.Client.FirefoxDetails = { accurate: false, version: "38.0", channel: "release", isUpToDate: false, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - download button visible (PASS)
Comment 9•9 years ago
|
||
Testing notes for Firefox 35.0 (Release)
----------------------------------------
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = true
Mozilla.Client.FirefoxDetails = { accurate: true, version: "35.0", channel: "release", isUpToDate: false, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - download button visible (PASS)
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = false
Mozilla.Client.FirefoxDetails = { accurate: false, version: "35.0", channel: "release", isUpToDate: false, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - download button visible (PASS)
Comment 10•9 years ago
|
||
Testing notes for Firefox 34.0 (Release)
----------------------------------------
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = true
Mozilla.Client.FirefoxDetails = { accurate: false, version: "34.0", channel: "release", isUpToDate: false, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - download button visible (PASS)
browser.uitour.testingOrigins = https://www-demo2.allizom.org/
browser.uitour.enabled = false
Mozilla.Client.FirefoxDetails = { accurate: false, version: "34.0", channel: "release", isUpToDate: false, isESR: false }
https://www-demo2.allizom.org/en-US/plugincheck/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/desktop/tips/ - download button visible (PASS)
https://www-demo2.allizom.org/en-US/firefox/products/ - download button visible (PASS)
Comment 11•9 years ago
|
||
So far, testing looks good thanks Kohei!
Comment 12•9 years ago
|
||
Gareth, would you be free to catch up either Friday or Monday to chat about GA for this change? I think we'll be ready to merge this one soon.
Flags: needinfo?(garethcull.bugs)
Comment 13•9 years ago
|
||
Sounds good. I've sent you an invite for tomorrow morning.
Flags: needinfo?(garethcull.bugs)
Comment 14•9 years ago
|
||
GA event to track:
window.dataLayer.push({
'event': 'firefox-details-retrieved',
'accurate': accurate,
'version': version,
'channel': channel,
'isUpToDate': isUpToDate,
'isESR': isESR
});
Comment 15•9 years ago
|
||
Commits pushed to master at https://github.com/mozilla/bedrock
https://github.com/mozilla/bedrock/commit/b262ae3b0ebe5e934887577369703e48fb5764d5
Fix Bug 1216225 - Add function to provide accurate Firefox channel/version info based on mozUITour API
https://github.com/mozilla/bedrock/commit/1584f9e386f6fd81f7c0634f31e2569c2c37a9ea
Merge pull request #3451 from kyoshino/bug-1216225-get-firefox-details
Bug 1216225 - Add function to provide accurate Firefox channel/version info based on mozUITour API
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 16•9 years ago
|
||
Gareth, now that this has been on prod for a little while, can you please take a look at the event data we discussed above for the 4 URLs?
Flags: needinfo?(garethcull.bugs)
Comment 17•9 years ago
|
||
Looks like this has dropped off the radar, Gareth can we please get an update on this one sometime soon? Thanks
Comment 18•9 years ago
|
||
Sorry for the delay on this Alex. I've added it to my next sprint and will get this to you next week.
Flags: needinfo?(garethcull.bugs)
You need to log in
before you can comment on or make changes to this bug.
Description
•