From a952148800a53857da1aaebab8cbd64fa9522d58 Mon Sep 17 00:00:00 2001
From: Jeff Escalante <gh.je@mailhero.io>
Date: Mon, 2 Apr 2018 12:30:15 -0400
Subject: [PATCH] adjustments to analytics scripts

- remove existing GTM tag from head to prevent double tracking
- adjust selector to not include non-download links
- send a cleaner breakdown of download parameters for easier analysis
---
 website/source/assets/javascripts/analytics.js | 9 +++++++--
 website/source/layouts/layout.erb              | 7 -------
 2 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/website/source/assets/javascripts/analytics.js b/website/source/assets/javascripts/analytics.js
index 33342909a..0e7dff179 100644
--- a/website/source/assets/javascripts/analytics.js
+++ b/website/source/assets/javascripts/analytics.js
@@ -1,9 +1,14 @@
 document.addEventListener('DOMContentLoaded', function() {
-  track('.downloads .download a', function(el) {
+  track('.downloads .download .details li a', function(el) {
+    var m = el.href.match(/packer_(\d+\.\d+\.\d+)_(.*?)_(.*?)\.zip/)
     return {
       event: 'Download',
       category: 'Button',
-      label: 'Packer | v' + el.href.match(/\/(\d+\.\d+\.\d+)\//)[1]
+      label: 'Packer | v' + m[1] + ' | ' + m[2] + ' | ' + m[3],
+      version: m[1],
+      os: m[2],
+      architecture: m[3],
+      product: 'packer'
     }
   })
 })
diff --git a/website/source/layouts/layout.erb b/website/source/layouts/layout.erb
index a632d884e..768ea0a83 100644
--- a/website/source/layouts/layout.erb
+++ b/website/source/layouts/layout.erb
@@ -34,13 +34,6 @@
     <![endif]-->
     <%= javascript_include_tag "application" %>
 
-    <!-- Google Tag Manager -->
-    <script>(function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
-    new Date().getTime(),event:'gtm.js'});var f=d.getElementsByTagName(s)[0],
-    j=d.createElement(s),dl=l!='dataLayer'?'&l='+l:'';j.async=true;j.src=
-    'https://www.googletagmanager.com/gtm.js?id='+i+dl;f.parentNode.insertBefore(j,f);
-    })(window,document,'script','dataLayer','GTM-NR2SD7C');</script>
-
     <!-- Typekit script to import Klavika font -->
     <script src="https://use.typekit.net/wxf7mfi.js"></script>
     <script>try{Typekit.load({ async: true });}catch(e){}</script>