chore(core): track otp usage (#10248)

close AF-2235
toeverything · Feb 18, 2025 · cd00f06 · cd00f06
1 parent cb15d48
commit cd00f06
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/packages/frontend/core/src/components/sign-in/sign-in-with-email.tsx b/packages/frontend/core/src/components/sign-in/sign-in-with-email.tsx
@@ -141,7 +141,7 @@ export const SignInWithEmailStep = ({
     setIsVerifying(true);
 
     try {
-      await authService.signInMagicLink(email, otp);
+      await authService.signInMagicLink(email, otp, false);
     } catch (e) {
       if (e instanceof BackendError) {
         notify.error({

diff --git a/packages/frontend/core/src/modules/cloud/services/auth.ts b/packages/frontend/core/src/modules/cloud/services/auth.ts
@@ -109,7 +109,8 @@ export class AuthService extends Service {
     }
   }
 
-  async signInMagicLink(email: string, token: string) {
+  async signInMagicLink(email: string, token: string, byLink = true) {
+    const method = byLink ? 'magic-link' : 'otp';
     try {
       await this.fetchService.fetch('/api/auth/magic-link', {
         method: 'POST',
@@ -120,10 +121,10 @@ export class AuthService extends Service {
       });
 
       this.session.revalidate();
-      track.$.$.auth.signedIn({ method: 'magic-link' });
+      track.$.$.auth.signedIn({ method });
     } catch (e) {
       track.$.$.auth.signInFail({
-        method: 'magic-link',
+        method,
         reason: e instanceof BackendError ? e.originError.name : 'unknown',
       });
       throw e;

diff --git a/packages/frontend/track/src/events.ts b/packages/frontend/track/src/events.ts
@@ -439,7 +439,7 @@ type SplitViewActionControlType = 'menu' | 'indicator';
 type SplitViewActionType = 'open' | 'close' | 'move' | 'closeOthers';
 
 type AuthArgs = {
-  method: 'password' | 'magic-link' | 'oauth';
+  method: 'password' | 'magic-link' | 'oauth' | 'otp';
   provider?: string;
 };