Merge pull request #306 from cfpb/fix_for_event_firing

Modified answer events to always fire irregardless of settlement status

CHANGELOG.md

@@ -1,6 +1,9 @@
 All notable changes to this project will be documented in this file.
 We follow the [Semantic Versioning 2.0.0](http://semver.org/) format.
 
+## 2.3.6
+- Modified Google Analytics question event to fire independent of settlement status
+
 ## 2.3.5
 - Fixed Google Analytics events on "Yes" and "Not sure" buttons
 

src/disclosures/js/views/question-view.js

@@ -80,11 +80,9 @@ var questionView = {
       } else if ( $( this ).attr( 'id' ) === 'question_answer-yes' ) {
         questionView.$followupYes.show();
         questionView.$followupNoNotSure.hide();
-        Analytics.sendEvent( getDataLayerOptions( 'Step Completed', 'Yes' ) );
       } else {
         questionView.$followupNoNotSure.show();
         questionView.$followupYes.hide();
-        Analytics.sendEvent( getDataLayerOptions( 'Step Completed', 'Not sure' ) );
       }
       // Show the rest of the page
       questionView.$getOptions.show();
@@ -95,6 +93,10 @@ var questionView = {
       }, 900, 'swing', function() {
         // Noop function.
       } );
+
+      Analytics.sendEvent( getDataLayerOptions( 'Step Completed',
+        $( this ).text().trim() )
+      );
     } );
   }