Remove overall contributors from retention on community metrics page

mythmon · Feb 29, 2016 · d4245f7 · d4245f7
1 parent 7270e79
commit d4245f7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/kitsune/community/templates/community/metrics.html b/kitsune/community/templates/community/metrics.html
@@ -10,13 +10,12 @@
 {% block content %}
   <article id="kpi-dash">
     <h1>{{ _('Contributor Metrics') }}</h1>
-      <section id="kpi-cohort-analysis" data-url="/api/2/cohort">
+      <section id="kpi-cohort-analysis" data-url="/api/2/cohort" data-contributor-type="contributor:kb:en-US">
       <h2>{{ _('Retention') }}</h2>
       <div id="retention-chart" class="graph" data-chart-type="d3" data-slug="retention">
         <div class="inline-controls">
           <div>
             <select id="toggle-cohort-type">
-              <option value="contributor">{{ _('Contributors') }}</option>
               <option value="contributor:kb:en-US">{{ _('en-US') }}</option>
               <option value="contributor:kb:l10n">{{ _('L10N') }}</option>
               <option value="contributor:supportforum:helper">{{ _('Support Forum Help') }}</option>

diff --git a/kitsune/kpi/static/kpi/js/kpi.browserify.js b/kitsune/kpi/static/kpi/js/kpi.browserify.js
@@ -244,6 +244,7 @@ function handleDataError($container) {
 function makeRetentionChart(settings) {
   let $container = settings.container;
   let startDate = moment().day(1).day(-84).format('YYYY-MM-DD');
+  let defaultContributorType = $container.data('contributor-type') || 'contributor';
   let urlToFetch = `${$container.data('url')}?start=${startDate}`;
   let fetchDataset = getChartData(urlToFetch, 'results');
   let retentionChart = new Chart($container, settings.options);
@@ -253,7 +254,7 @@ function makeRetentionChart(settings) {
     retentionChart.data = data;
     retentionChart.axes.yAxis.labels = _.uniq(_.pluck(data, 'start'));
     retentionChart.setupAxis('yAxis');
-    retentionChart.populateData('contributor');
+    retentionChart.populateData(defaultContributorType);
 
     $('#toggle-cohort-type').change(function() {
       let cohortType = $(this).val();