PL: teacher application posts userId to firehose when starting

Author: breville

apps/src/code-studio/pd/application/teacher1920/Teacher1920Application.jsx

@@ -12,7 +12,8 @@ import firehoseClient from "@cdo/apps/lib/util/firehose";
 export default class Teacher1920Application extends FormController {
   static propTypes = {
     ...FormController.propTypes,
-    accountEmail: PropTypes.string.isRequired
+    accountEmail: PropTypes.string.isRequired,
+    userId: PropTypes.number.isRequired
   };
 
   static submitButtonText = "Complete and Send";
@@ -50,10 +51,11 @@ export default class Teacher1920Application extends FormController {
     // Log the user ID to firehose.
     firehoseClient.putRecord(
       {
+        userId: this.props.userId,
         study: 'application-funnel',
         event: 'started-teacher1920-application'
       },
-      {includeUserId: true}
+      {includeUserId: false}
     );
   }
 

dashboard/app/controllers/pd/application/teacher_application_controller.rb

@@ -23,7 +23,8 @@ def new
           options: TEACHER_APPLICATION_CLASS.options.camelize_keys,
           requiredFields: TEACHER_APPLICATION_CLASS.camelize_required_fields,
           accountEmail: current_user.email,
-          apiEndpoint: '/api/v1/pd/application/teacher'
+          apiEndpoint: '/api/v1/pd/application/teacher',
+          userId: current_user.id
         }.to_json
       }
     end