Merge pull request #6 from datainsider-co/upgrade_tracking

update: version of di analytics to 0.8.6
datainsider-co · Jan 16, 2023 · be5b652 · be5b652
2 parents 2c2b063 + 73341a1
commit be5b652
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "di-web-analytics",
-  "version": "0.8.5",
+  "version": "0.8.6",
   "description": "Data insider web analytics",
   "repository": {
     "type": "git",

diff --git a/src/service/analytics_core_impl.ts b/src/service/analytics_core_impl.ts
@@ -47,10 +47,8 @@ export class AnalyticsCoreImpl extends AnalyticsCore {
 
   private setupWorker() {
 
-    document.addEventListener('readystatechange', event => {
-      window.addEventListener('beforeunload', async (event) => {
-        await this.worker.stop();
-      });
+    window.addEventListener('beforeunload', async (event) => {
+      await this.worker.stop();
     });
   }
 

diff --git a/src/service/di_analytics.ts b/src/service/di_analytics.ts
@@ -170,11 +170,11 @@ export class DiAnalytics {
     properties?: Properties,
   ): Promise<void> {
     try {
-      await this.identify(customerInfo.id);
       await this.track(SystemEvents.Login, {
         ...customerInfo,
         ...properties
       });
+      await this.identify(customerInfo.id);
     } catch (ex) {
       Logger.error('DiAnalytics.login failed', ex);
     }