improve data tracker (#114)

domechn · Aug 5, 2023 · 8cb990d · 8cb990d
1 parent 41d8bb1
commit 8cb990d
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 14 deletions.
diff --git a/src/components/data-management/index.tsx b/src/components/data-management/index.tsx
@@ -21,6 +21,7 @@ import {
 } from "../../middlelayers/cloudsync";
 import { LoadingContext } from "../../App";
 import { timestampToDate } from "../../utils/date";
+import { trackEventWithClientID } from '../../utils/app'
 
 const App = ({
   onDataImported,
@@ -139,6 +140,8 @@ const App = ({
       }
       try {
         await signIn(email, verificationCode);
+
+        await trackEventWithClientID("sign_in")
       } finally {
         if (signInRef.current) {
           signInRef.current!.disabled = false;

diff --git a/src/components/refresh-data/index.tsx b/src/components/refresh-data/index.tsx
@@ -5,7 +5,7 @@ import { refreshAllData } from "../../middlelayers/charts";
 import { toast } from "react-hot-toast";
 import { LoadingContext } from "../../App";
 import { updateAllCurrencyRates } from "../../middlelayers/currency";
-import { trackEvent } from '@aptabase/tauri'
+import { trackEventWithClientID } from '../../utils/app'
 
 const retries = 3;
 const retryInterval = 3000; // 3s
@@ -51,7 +51,7 @@ const App = ({
       })
       .finally(() => {
         setLoading(false);
-        trackEvent("data_refreshed")
+        trackEventWithClientID("data_refreshed")
         if (refreshError) {
           toast.error(refreshError.message || (refreshError as any));
         } else {

diff --git a/src/main.tsx b/src/main.tsx
@@ -2,8 +2,7 @@ import React from "react";
 import ReactDOM from "react-dom/client";
 import App from "./App";
 import "./style.css";
-import { getClientID, getVersion } from "./utils/app";
-import { trackEvent } from "@aptabase/tauri";
+import { trackEventWithClientID } from "./utils/app";
 
 ReactDOM.createRoot(document.getElementById("root") as HTMLElement).render(
   <React.StrictMode>
@@ -27,13 +26,5 @@ function disableContextmenu() {
 
 disableContextmenu();
 
-// ga4
-(async () => {
-  try {
-    const cid = await getClientID();
-    trackEvent("app_started", { clientId: cid || "unknown" });
-  } catch (e) {
-    trackEvent("app_started");
-    throw e;
-  }
-})();
+// track event
+(async () => trackEventWithClientID("app_started"))();
diff --git a/src/utils/app.ts b/src/utils/app.ts
@@ -1,5 +1,6 @@
 import * as api from '@tauri-apps/api'
 import { getClientIDConfiguration } from '../middlelayers/configuration'
+import { trackEvent } from '@aptabase/tauri'
 
 export async function getVersion() {
 	return api.app.getVersion()
@@ -8,3 +9,16 @@ export async function getVersion() {
 export async function getClientID() {
 	return getClientIDConfiguration()
 }
+
+
+export async function trackEventWithClientID(event: string, props?: { [k: string]: string | number }) {
+	const cid = await getClientID()
+	try {
+		await trackEvent(event, {
+			clientID: cid || "unknown",
+			...(props ?? {})
+		})
+	} catch (e) {
+		console.error("track event failed", e)
+	}
+}