Only send JS errors+stats in production [no merge]

NEWS.md

@@ -18,6 +18,7 @@
 * Added new public pages [#2034](https://github.com/CartoDB/cartodb/pull/2142)
 * Added API keys and OAuth pages [#2142](https://github.com/CartoDB/cartodb/pull/2142)
 * Replace 404 error page
+* Only send JS errors+stats in production [#1987](https://github.com/CartoDB/cartodb/pull/1987)
 
 Bugfixes:
 * When being in any configuration page remove the arrow from the breadcrumb [#2312](https://github.com/CartoDB/cartodb/pull/2312)

app/helpers/application_helper.rb

@@ -156,7 +156,7 @@ def insert_google_analytics(track, custom_vars = {})
 
   def insert_trackjs
     if not Cartodb.config[:trackjs].blank? and not Cartodb.config[:trackjs]['customer'].blank?
-      render(:partial => 'shared/trackjs', :locals => { customer: Cartodb.config[:trackjs]['customer'] })
+      render(:partial => 'shared/trackjs', :locals => { customer: Cartodb.config[:trackjs]['customer'], enabled: Cartodb.config[:trackjs]['enabled'] })
     end
   end
 

app/views/shared/_trackjs.html.erb

@@ -1,3 +1,6 @@
-<% unless customer.blank? || Rails.env.development? %>
+<% unless customer.blank? %>
+  <script>
+    window._trackJs = { enabled: <%= enabled ? 'true' : 'false' %> };
+  </script>
   <script type="text/javascript" src="//d2zah9y47r7bi2.cloudfront.net/releases/current/tracker.js" data-token="<%= customer %>"></script>
-<% end %>
+<% end %>

config/app_config.yml.sample

@@ -49,6 +49,7 @@ defaults: &defaults
   tumblr:
     api_key: ''
   trackjs:
+    enabled: false
     customer: ''
   common_data:
        protocol: 'https'

lib/assets/javascripts/cartodb/new_dashboard/entry.js

@@ -2,12 +2,13 @@ var Router = require('new_dashboard/router');
 var $ = require('jquery');
 var cdb = require('cartodb.js');
 var MainView = require('new_dashboard/main_view');
-var trackJsErrors = require('new_common/track_js_errors');
 var sendUsageToMixpanel = require('./send_usage_to_mixpanel');
 var urls = require('new_common/urls_fn');
 
 if (window.trackJs) {
-  trackJsErrors(window.trackJs, window.user_data.username);
+  window.trackJs.configure({
+    userId: window.user_data.username
+  });
 }
 
 /**

lib/assets/javascripts/cartodb/new_keys/entry.js

@@ -1,6 +1,5 @@
 var $ = require('jquery');
 var cdb = require('cartodb.js');
-var trackJsErrors = require('new_common/track_js_errors');
 var HeaderView = require('new_common/views/dashboard_header_view');
 var SupportView = require('new_common/support_view');
 var HeaderViewModel = require('new_keys/header_view_model');
@@ -9,7 +8,9 @@ var urls = require('new_common/urls_fn');
 var RegenerateKeysDialog = require('new_keys/regenerate_keys_dialog_view');
 
 if (window.trackJs) {
-  trackJsErrors(window.trackJs, window.user_data.username);
+  window.trackJs.configure({
+    userId: window.user_data.username
+  });
 }
 
 /**