Merge pull request #89 from Genymobile/dev/fix-gapps-installed-track-…

…events-send-too-many-times send gapps only after installation
Genymobile · Aug 6, 2024 · e9eca0a · e9eca0a
2 parents 834a163 + 1adede6
commit e9eca0a
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/plugins/FileUpload.js b/src/plugins/FileUpload.js
@@ -277,13 +277,6 @@ module.exports = class FileUpload extends OverlayPlugin {
             textProgress.innerHTML = this.i18n.UPLOADER_INSTALLING || 'Installing...';
         } else if (data[0] === 'ready' && data.length >= 2) {
             if (data[1].indexOf('opengapps') !== -1) {
-                this.instance.store.dispatch({
-                    type: 'ADD_TRACKED_EVENT',
-                    payload: {
-                        category: 'opengapps',
-                        type: 'installed',
-                    },
-                });
                 this.updateOpenGAppsStatus(true);
             }
 
@@ -313,6 +306,13 @@ module.exports = class FileUpload extends OverlayPlugin {
                     this.toggleWidget();
                 }
                 this.flashing = false;
+                this.instance.store.dispatch({
+                    type: 'ADD_TRACKED_EVENT',
+                    payload: {
+                        category: 'opengapps',
+                        action: 'installed',
+                    },
+                });
                 break;
             case 'unavailable':
             case 'network_error':