Merge pull request #20943 from code-dot-org/log-main-json-source-url

also log sourceUrl as it appears in main.json
code-dot-org · Feb 28, 2018 · 5cb89f3 · 5cb89f3
2 parents 941b024 + 0e53cd2
commit 5cb89f3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/apps/src/gamelab/ErrorDialogStack.jsx b/apps/src/gamelab/ErrorDialogStack.jsx
@@ -30,7 +30,7 @@ class ErrorDialogStack extends React.Component {
       'analysis-events',
       {
         study: 'animation_no_load',
-        study_group: 'animation_no_load_v3',
+        study_group: 'animation_no_load_v4',
         event: 'delete_selected',
         project_id: getCurrentId(),
         data_json: JSON.stringify({'version': this.props.animationList.propsByKey[key].version,
@@ -48,7 +48,7 @@ class ErrorDialogStack extends React.Component {
       'analysis-events',
       {
         study: 'animation_no_load',
-        study_group: 'animation_no_load_v3',
+        study_group: 'animation_no_load_v4',
         event: 'reload_selected',
         project_id: getCurrentId(),
         data_json: JSON.stringify({'version': this.props.animationList.propsByKey[key].version,

diff --git a/apps/src/gamelab/animationListModule.js b/apps/src/gamelab/animationListModule.js
@@ -570,11 +570,16 @@ function loadAnimationFromSource(key, callback) {
          'analysis-events',
           {
             study: 'animation_no_load',
-            study_group: 'animation_no_load_v3',
+            study_group: 'animation_no_load_v4',
             event: isOwner() ? 'animation_not_loaded_owner' : 'animation_not_loaded_viewer',
             project_id: getCurrentId(),
-            data_json: JSON.stringify({'sourceUrl': sourceUrl, 'version': state.propsByKey[key].version,
-              'animationName': state.propsByKey[key].name, 'error': err.message})
+            data_json: JSON.stringify({
+              'sourceUrl': sourceUrl,
+              'mainJsonSourceUrl': state.propsByKey[key].sourceUrl,
+              'version': state.propsByKey[key].version,
+              'animationName': state.propsByKey[key].name,
+              'error': err.message
+            })
           },
           {includeUserId: true}
         );