Skip to content
This repository has been archived by the owner on Oct 2, 2018. It is now read-only.

Commit

Permalink
Revert "Added Google Analytics"
Browse files Browse the repository at this point in the history 
This reverts commit f0e4a4f.
  • Loading branch information
Joshua Smith committed Jul 7, 2013
1 parent 4836f90 commit d787c6e
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions index.html
View file
Original file line number Diff line number Diff line change
Expand Up @@ -39,18 +39,6 @@
<link rel="stylesheet" type="text/css" href="style/regions.css"/>
<link rel="stylesheet" type="text/css" href="resources/google-code-prettify/prettify.css"/>

<!-- Analytics -->
<script type="text/javascript">
var _gaq = _gaq || [];
_gaq.push(['_setAccount', 'UA-41224040-1']);
_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>

<!-- Shared code -->
<script type="application/javascript" defer src="js/gesture_detector.js"></script>

Expand Down

4 comments on commit d787c6e

@ferndot
Copy link
Member

@ferndot ferndot commented on d787c6e Jul 7, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This created a major csp error, sorry!

@HR
Copy link
Member

@HR HR commented on d787c6e Jul 7, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove this, we need this vitally for tracking and reporting so that we can enhance UX and introduce new layouts to make the user interaction better as well as other stuff...

@HR
Copy link
Member

@HR HR commented on d787c6e Jul 7, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm gonna resolve this later. But we can't release it without Google analytics

@ferndot
Copy link
Member

@ferndot ferndot commented on d787c6e Jul 7, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to, Google Analytics are fine for the online version, but it invalidates our privacy policy, and might block approval. Anyway, inline or remote scripts are banned.

Please sign in to comment.