Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update documentation regarding Google Analytics #1515

Closed
wants to merge 1 commit into from
Closed

Update documentation regarding Google Analytics #1515

wants to merge 1 commit into from

Conversation

tomfuertes
Copy link
Contributor

Needed some updating :)

Ref: #1490.

@alrra
Copy link
Member

alrra commented Mar 1, 2014

Thanks @tomfuertes!

Can you also update the information from html.md (update links, specify that we're talking about the Google Universal Analytics snippet, etc.)?

@alrra alrra added this to the 5.0 milestone Mar 1, 2014
@alrra alrra self-assigned this Mar 1, 2014
@tomfuertes
Copy link
Contributor Author

Updated + Rebased

snippet](http://mathiasbynens.be/notes/async-analytics-snippet).
* [Tracking Site Activity - Google
Analytics](http://code.google.com/apis/analytics/docs/tracking/asyncTracking.html).
* ["Advanced" Setup](https://developers.google.com/analytics/devguides/collection/analyticsjs/advanced)

This comment was marked as abuse.

@tomfuertes
Copy link
Contributor Author

Space Added + Links Updated

@alrra
Copy link
Member

alrra commented Mar 1, 2014

Can you also update the information from html.md (update links, specify that we're talking about the Google Universal Analytics snippet, etc.)?

@tomfuertes I've missed: usage.md#indexhtml (needs to be Google Universal Analytics)

Updated + Rebased
Space Added + Links Updated

@tomfuertes you're awesome! thanks!

@tomfuertes
Copy link
Contributor Author

Updated + Rebased

@@ -83,8 +83,8 @@ need to integrate this starting HTML with your setup.
Make sure that you update the URLs for the referenced CSS and JavaScript if you
modify the directory structure at all.

If you are using Google Analytics, make sure that you edit the corresponding
snippet at the bottom to include your analytics ID.
If you are using Google's Universal Analytics, make sure that you edit the

This comment was marked as abuse.

@tomfuertes
Copy link
Contributor Author

Addressed all above

`_gat.anonymizeIp`
option](http://code.google.com/apis/analytics/docs/gaJS/gaJSApi_gat.html#_gat._anonymizeIp).
`'anonymizeIp': true`
option](https://developers.google.com/analytics/devguides/collection/analyticsjs/advanced#anonymizeip).

This comment was marked as abuse.

This comment was marked as abuse.

@alrra
Copy link
Member

alrra commented Mar 1, 2014

@tomfuertes left some more (probably final) comments.

edit: updated title.

@alrra alrra removed their assignment Mar 2, 2014
alrra pushed a commit that referenced this pull request Mar 2, 2014
@alrra
Copy link
Member

alrra commented Mar 2, 2014

Merged (with some minor modifications) in 7997c1b.

Thanks @tomfuertes!

@alrra alrra closed this Mar 2, 2014
alrra pushed a commit that referenced this pull request Mar 20, 2014
eleanor-byhook pushed a commit to eleanor-byhook/html5-boilerplate that referenced this pull request Feb 29, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants