Closed
Bug 840269
Opened 13 years ago
Closed 13 years ago
SOPA: Replace GA code at the bottom with a PHP include at the top
Categories
(www.mozilla.org :: Legacy PHP system, defect)
www.mozilla.org
Legacy PHP system
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: hoosteeno, Assigned: hoosteeno)
References
Details
(Whiteboard: u=admin c=analytics p= r=113718 )
On the Stop SOPA page, can we replace the Webtrends code at the bottom of the page with a Google Analytics PHP include at the top?
The page (http://www.mozilla.org/sopa/) is here:
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/en-US/sopa/index.html?view=markup
The PHP to include is here:
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/includes/google-analytics.php?view=markup
The code should be included immediately before </head>.
Comment 1•13 years ago
|
||
We can get this on the next Mozilla.org sprint or see if anyone has any bandwidth to get it in sometime soon.
Updated•13 years ago
|
Whiteboard: u=admin c=analytics p=
| Assignee | ||
Comment 2•13 years ago
|
||
I just committed revision 113718 with code that I hope will address this bug.
Assignee: nobody → hoosteeno
Keywords: qawanted
Whiteboard: u=admin c=analytics p= → u=admin c=analytics p= r=113718
Comment 4•13 years ago
|
||
I'm not seeing this on trunk www.allizom.org/sopa/
Comment 5•13 years ago
|
||
https://www-dev.allizom.org/sopa/ I see it.
| Assignee | ||
Comment 6•13 years ago
|
||
The addition starts on line 307 in source. The subtraction was down around 400-410.
Comment 7•13 years ago
|
||
This is fixed on stage https://www-dev.allizom.org/sopa/
Comment 8•13 years ago
|
||
Committed to prod in r113722.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•