Add metric (+fix) (#15400)

packages/twenty-server/src/engine/core-modules/auth/services/sign-in-up.service.spec.ts

@@ -20,10 +20,11 @@ import {
 import { SubdomainManagerService } from 'src/engine/core-modules/domain/subdomain-manager/services/subdomain-manager.service';
 import { FeatureFlagService } from 'src/engine/core-modules/feature-flag/services/feature-flag.service';
 import { FileUploadService } from 'src/engine/core-modules/file/file-upload/services/file-upload.service';
+import { MetricsService } from 'src/engine/core-modules/metrics/metrics.service';
 import { OnboardingService } from 'src/engine/core-modules/onboarding/onboarding.service';
 import { TwentyConfigService } from 'src/engine/core-modules/twenty-config/twenty-config.service';
-import { UserWorkspaceService } from 'src/engine/core-modules/user-workspace/user-workspace.service';
 import { type UserWorkspaceEntity } from 'src/engine/core-modules/user-workspace/user-workspace.entity';
+import { UserWorkspaceService } from 'src/engine/core-modules/user-workspace/user-workspace.service';
 import { UserService } from 'src/engine/core-modules/user/services/user.service';
 import { UserEntity } from 'src/engine/core-modules/user/user.entity';
 import { WorkspaceInvitationService } from 'src/engine/core-modules/workspace-invitation/services/workspace-invitation.service';
@@ -143,6 +144,12 @@ describe('SignInUpService', () => {
             emitCustomBatchEvent: jest.fn(),
           },
         },
+        {
+          provide: MetricsService,
+          useValue: {
+            incrementCounter: jest.fn(),
+          },
+        },
       ],
     }).compile();
 

packages/twenty-server/src/engine/core-modules/auth/services/sign-in-up.service.ts

@@ -27,6 +27,8 @@ import {
   type SignInUpNewUserPayload,
 } from 'src/engine/core-modules/auth/types/signInUp.type';
 import { SubdomainManagerService } from 'src/engine/core-modules/domain/subdomain-manager/services/subdomain-manager.service';
+import { MetricsService } from 'src/engine/core-modules/metrics/metrics.service';
+import { MetricsKeys } from 'src/engine/core-modules/metrics/types/metrics-keys.type';
 import { OnboardingService } from 'src/engine/core-modules/onboarding/onboarding.service';
 import { TwentyConfigService } from 'src/engine/core-modules/twenty-config/twenty-config.service';
 import { UserWorkspaceService } from 'src/engine/core-modules/user-workspace/user-workspace.service';
@@ -55,6 +57,7 @@ export class SignInUpService {
     private readonly twentyConfigService: TwentyConfigService,
     private readonly subdomainManagerService: SubdomainManagerService,
     private readonly userService: UserService,
+    private readonly metricsService: MetricsService,
   ) {}
 
   async computePartialUserFromUserPayload(
@@ -339,6 +342,11 @@ export class SignInUpService {
       undefined,
     );
 
+    this.metricsService.incrementCounter({
+      key: MetricsKeys.SignUpSuccess,
+      shouldStoreInCache: false,
+    });
+
     return savedUser;
   }
 

packages/twenty-server/src/engine/core-modules/metrics/types/metrics-keys.type.ts

@@ -28,4 +28,5 @@ export enum MetricsKeys {
   AppVersionMismatch = 'app-version/mismatch',
   CronJobDeletedWorkspace = 'cron-job/deleted-workspace',
   JobWebhookCallCompleted = 'job/webhook-call-completed',
+  SignUpSuccess = 'sign-up/success',
 }