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)
www.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: stas, Assigned: stas)
References
Details
Attachments
(2 files, 3 obsolete files)
8.57 KB,
text/html
|
Details | |
741 bytes,
patch
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 2•17 years ago
|
||
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.
Assignee | ||
Comment 3•17 years ago
|
||
The script addresses both bug 393056 and bug 391347. A testcase follows.
Attachment #278565 -
Flags: review?(clouserw)
Assignee | ||
Comment 4•17 years ago
|
||
A testcase.
Assignee | ||
Comment 5•17 years ago
|
||
CC'ing marcoos: could you please test the script with Safari 2?
Assignee | ||
Comment 6•17 years ago
|
||
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.
Assignee | ||
Comment 7•17 years ago
|
||
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 :-)
Assignee | ||
Comment 8•17 years ago
|
||
The script works correctly in Safari 2 (as tested on Tristan's Mac).
Comment 9•17 years ago
|
||
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+
Assignee | ||
Comment 10•17 years ago
|
||
OK. The script is ready to be checked-in :-)
Comment 11•17 years ago
|
||
Script is on trunk, r6376.
Assignee | ||
Comment 12•17 years ago
|
||
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 13•17 years ago
|
||
Attachment #279123 -
Flags: review?(clouserw) → review+
Updated•17 years ago
|
Assignee: nobody → smalolepszy
Assignee | ||
Comment 14•17 years ago
|
||
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
Comment 15•17 years ago
|
||
> 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
Assignee | ||
Comment 16•17 years ago
|
||
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
Assignee | ||
Comment 17•17 years ago
|
||
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.
Comment 18•17 years ago
|
||
patch from comment 16 is on trunk, r6626
Assignee | ||
Comment 19•17 years ago
|
||
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?
Assignee | ||
Comment 20•17 years ago
|
||
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!
Comment 21•17 years ago
|
||
(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.
Assignee | ||
Comment 22•17 years ago
|
||
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 :-)
Comment 23•17 years ago
|
||
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.
Assignee | ||
Comment 24•17 years ago
|
||
It's gone live. Thank you all for your help!
Resolved fixed.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 25•15 years ago
|
||
Heads-up: Urchin (and therefore track.js) is going away on mozilla.com shortly. See bug 556384.
Updated•13 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
Updated•12 years ago
|
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.
Description
•