Closed Bug 392342 Opened 18 years ago Closed 17 years ago

Ensure traffic flow data between different Mozilla websites is not lost

Categories

(www.mozilla.org :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: stas, Assigned: stas)

References

Details

Attachments

(2 files, 3 obsolete files)

This bug will track progress of adjusting the Mozilla websites so that traffic flow data between our different websites is not lost. E.g. when a user browses away from mozilla.com to mozilla-europe.org, we get some data about his two visits in two Urchin profiles, but with no possibility (to the best of my knowledge) to link it in one interpretable piece.
Let's break this down into smaller pieces: 1. how many users come to Firefox product pages via the getfirefox.com redirection? -- bug 393052. 2. how do users behave on the in-product pages on mozilla.com (firstrun, whatsnew, central, and others); do they click the links we've put on them? -- bug 393056. 3. how many downloads are there from the mozilla.com pages (e.g. all.html)? See bug 391347 for this.
Depends on: 391347, 393052, 393056
Depends on: 393200
The script addresses both bug 393056 and bug 391347. A testcase follows.
Attachment #278565 - Flags: review?(clouserw)
A testcase.
CC'ing marcoos: could you please test the script with Safari 2?
I tested with success using: - Firefox 2.0.0.6 (returns: using:firefox2) - Safari 3 beta (returns: using:safari) - IE 6 sp2 (returns: using:ie6) - Opera 9.20 (returns: using:opera) - SeaMonkey 1.1.4 (returns: using:gecko-other) - K-meleon 1.1 (returns: using:gecko-other) All on Windows XP SP2. We need tests for IE7 and Safari 2.
As Pascal suggested IRL, we may want to change the browser detection to detect Firefox3, by adding the following line: + else if (n.indexOf("Firefox/3") != -1) return "firefox3"; else if (n.indexOf("Firefox/2") != -1) return "firefox2"; This would spare us some work in November :-)
The script works correctly in Safari 2 (as tested on Tristan's Mac).
Comment on attachment 278565 [details] Script adding trackers to chosen links (oownload and outbound) irefox3"; > else if (n.indexOf("Firefox/2") != -1) return "firefox2"; I think that is a good idea. I've tested in ie7+xp.
Attachment #278565 - Flags: review?(clouserw) → review+
Attached file Added Firefox 3 detection line (obsolete) —
OK. The script is ready to be checked-in :-)
Script is on trunk, r6376.
Attached patch Exluding en-US links (obsolete) — Splinter Review
After some testing on stage, I have updated the script, so that it doesn't add to event listeners for certain links. I've changed the installClickTrackers() function: * removed the last argument (boolean) * added to regular expression as arguments: first one matches all the links to add the tracker to, the second one matches the ones to ecxlude. Wil, please review the change and apply the patch if r+. I can test it on stage then, and we could publish it soon, to start collecting data. Thanks!
Attachment #279123 - Flags: review?(clouserw)
Comment on attachment 279123 [details] [diff] [review] Exluding en-US links in svn, r6465
Attachment #279123 - Flags: review?(clouserw) → review+
Assignee: nobody → smalolepszy
I've tested the script on stage and it seemed to work fine. However, I only tested at mozilla.com/en-US. Other languages are only present for the in-product pages. Wil, would you agree to put in includes/header-portal-pages.inc.php as well, so that all the in-product pages can use it? It was one of my intentions, to learn how our users behave after they install / upgrade Firefox. For example, do they follow the support links we propose, or are they interested in visiting AMO.
Status: NEW → ASSIGNED
> Wil, would you agree to put in includes/header-portal-pages.inc.php as well, so > that all the in-product pages can use it? It was one of my intentions, to learn > how our users behave after they install / upgrade Firefox. For example, do they > follow the support links we propose, or are they interested in visiting AMO. > Done, r6566
I've noticed that the URLs like pl.www.mozilla.com/pl/products don't redirect to mozilla-europe (as /support links do). Instead they redirect to en-US/products, so there is no need of setting up additional urchinTrackers here. On the other hand, the release notes links present on the whatsnew page do redirect to mozilla-europe for non-American locales. The change in this patch reflect these two facts.
Attachment #278565 - Attachment is obsolete: true
Attachment #278791 - Attachment is obsolete: true
Attachment #279123 - Attachment is obsolete: true
If anyone wants to test it thoroughly, please have a look at http://wiki.mozilla.org/User:Staszyk/Testing_track.js_on_stage where I wrote down a few instructions for testing. Thanks.
patch from comment 16 is on trunk, r6626
OK, I tested the script a couple of times, and so did Pascal on his machine. Everything works fine. Wil, would you publish it on the public site, please?
Reed, can I ask you to merge rev.6626 with the production tag, so as to include the patch from comment 16 on the live site? Thanks!
(In reply to comment #20) > Reed, can I ask you to merge rev.6626 with the production tag, so as to include > the patch from comment 16 on the live site? Sending production/js/track.js Transmitting file data . Committed revision 6716.
Reed, thanks! I'm sorry to bother you again, but would you merge rev. 6566 as well, please? It's the /includes/header-portal-pages.inc.php file. (http://viewvc.svn.mozilla.org/vc?view=rev&revision=6566) After this merge the bug's going to be resolved :-)
Sending production/includes/header-portal-pages.inc.php Transmitting file data . Committed revision 6722. This won't go live until the next push of mozilla.com.
It's gone live. Thank you all for your help! Resolved fixed.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Heads-up: Urchin (and therefore track.js) is going away on mozilla.com shortly. See bug 556384.
Component: www.mozilla.org/firefox → www.mozilla.org
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: