816309 - Move Mozillians From Webtrends To Google Analytics

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[Ben (:bensternthal)]

Move Mozillians From Webtrends To Google Analytics

Comment 1

[William Reynolds [:williamr]]

Directions can be found here:
https://intranet.mozilla.org/Webanalytics/Google_Analytics#Adding_Google_Analytics_code_to_your_website

Comment 2

[Ben (:bensternthal)]

Chris, I can take this one since I would also be handling the implementation on the site.

Let me know if it's good to go and if it would be a "big" or "small"

Comment 3

[Chris More [:cmore]]

This should be a small and you can take it over.

Comment 4

[Ben (:bensternthal)]

OK, GA tag is in #821386

Giorgos, you might want to check out the implementation on bedrock, they are storing the GA id in settings.

https://github.com/mozilla/bedrock/blob/master/templates/includes/google-analytics.html

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/22b89b5bd1c7d48ad6c9fc58f64a95921277920c
[fix Bug 816309] Move Mozillians from Webtrends to Google Analytics

https://github.com/mozilla/mozillians/commit/31dbd11a5c19bc7a88b3c3804dd36b4c9f0a4164
Merge pull request #377 from Sancus/googleanalytics

[fix Bug 816309] Move Mozillians from Webtrends to Google Analytics

Comment 6

[Chris More [:cmore]]

This should be the correct code for the website:

<script type="text/javascript">

  var _gaq = _gaq || [];
  _gaq.push(['_setAccount', 'UA-35433268-19']);
  _gaq.push(['_trackPageview']);

  (function() {
    var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
    ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
    var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);
  })();

</script>

Comment 7

[Justin Crawford [:hoosteeno] [:jcrawford]]

It looks like the commit didn't include changes to the settings file (where the account code would go).

We just need to create the setting and ask IT to populate it after push. The account ID is in comment 6 above.

Comment 8

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/05c8e7cdcdfcad319d19d52c1bb8dcc887ca1b54
[fix Bug 816309] Add Google Analytics account code to settings

https://github.com/mozilla/mozillians/commit/db4d0e58598239b80537775fee7a747adeb358b4
Merge pull request #396 from Sancus/master

refix bug 816309: Add Google Analytics code to settings

Comment 9

[[github robot]]

Commit pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/0ac7a9e584bca315a4948209813f5095bf34f7c7
[bug 816309] Set GA_ACCOUNT_CODE in local.py-devdist.

Comment 10

[Matt Brandt [:mbrandt]]

Bumping to verified as [qa-]