Merge pull request #1026 from baylee-d/feature/add-collection-provider-to-analytics

[ENG-1135] Update collections analytics to include provider

Author: baylee-d

lib/app-components/addon/components/branded-navbar/template.hbs

@@ -43,7 +43,7 @@
                     <li>
                         {{#link-to
                             'provider.submit'
-                            click=(action 'click' 'link' (concat 'Navbar - ' (t @addLinkKey)) target=this.analytics)
+                            click=(action 'click' 'link' (concat 'Navbar - ' (t @addLinkKey) ' - ' this.theme.id) target=this.analytics)
                         }}
                             <span role='button'>{{t @addLinkKey}}</span>
                         {{/link-to}}

lib/collections/addon/components/collection-search-result/node/template.hbs

@@ -1,6 +1,6 @@
 <div
     data-test-collection-search-result-node={{this.item.id}}
-    data-analytics-scope='Collections search results'
+    data-analytics-scope={{concat 'Collections search results - ' this.theme.id}}
     class='row'
 >
     <div class='col-xs-12'>

lib/collections/addon/components/discover-page/component.ts

@@ -338,8 +338,7 @@ export default class DiscoverPage extends Component {
 
     @action
     clearFilters() {
-        this.analytics.track('button', 'click', 'Discover - Clear Filters');
-
+        this.analytics.track('button', 'click', `Discover - Clear Filters - ${this.theme.id}`);
         if (this.facetContexts) {
             // Clear all of the activeFilters
             this.facetContexts
@@ -366,7 +365,7 @@ export default class DiscoverPage extends Component {
     @action
     searchAction() {
         // Only want to track search here when button clicked. Keypress search tracking is debounced in trackSearch
-        this.analytics.track('button', 'click', 'Discover - Search', this.q);
+        this.analytics.track('button', 'click', `Discover - Search - ${this.theme.id}`, this.q);
         this.search();
     }
 

lib/collections/addon/components/discover-page/facets/checklist/component.ts

@@ -74,7 +74,7 @@ export default abstract class SearchFacetChecklist extends Base {
     didInsertElement(this: SearchFacetChecklist) {
         super.didInsertElement();
 
-        const { analytics, context, filterChanged, filterProperty } = this;
+        const { analytics, context, filterChanged, filterProperty, theme } = this;
 
         setProperties(context, {
             updateFilters(item?: string) {
@@ -84,7 +84,11 @@ export default abstract class SearchFacetChecklist extends Base {
                     const method = activeFilter.includes(item) ? 'removeObject' : 'pushObject';
                     activeFilter[method](item);
                     const filterAction = method === 'removeObject' ? 'remove' : 'add';
-                    analytics.track('filter', filterAction, `Collections Discover - Filter ${context.title} ${item}`);
+                    analytics.track(
+                        'filter',
+                        filterAction,
+                        `Discover - Filter ${context.title} ${item} - ${theme.id}`,
+                    );
                 }
 
                 setProperties(context, {