Default user status to Active (#3520)

Author: CarsonF

src/components/user/dto/create-person.dto.ts

@@ -33,7 +33,7 @@ export abstract class CreatePerson {
   readonly about?: string | null;
 
   @Field(() => UserStatus, { nullable: true })
-  readonly status?: UserStatus;
+  readonly status?: UserStatus = 'Active';
 
   @Field(() => [Role], { nullable: true })
   @Transform(({ value }) => uniq(value))

src/components/user/migrations/default-user-status.migration.ts

@@ -0,0 +1,14 @@
+import { BaseMigration, Migration } from '~/core/database';
+import { UserStatus } from '../dto';
+
+@Migration('2025-07-02T08:00:00')
+export class DefaultUserStatusMigration extends BaseMigration {
+  async up() {
+    await this.db.query().raw`
+      match (:User)-[:status]->(prop)
+      where prop.value is null
+      set prop.value = ${UserStatus.Active}
+      return count(prop)
+    `.executeAndLogStats();
+  }
+}

src/components/user/user.module.ts

@@ -13,6 +13,7 @@ import { KnownLanguageRepository } from './known-language.repository';
 import { KnownLanguageResolver } from './known-language.resolver';
 import { AddActorLabelMigration } from './migrations/add-actor-label.migration';
 import { AddUserNameLabelMigration } from './migrations/add-user-name-label.migration';
+import { DefaultUserStatusMigration } from './migrations/default-user-status.migration';
 import { SystemAgentGelRepository } from './system-agent.gel.repository';
 import { SystemAgentNeo4jRepository } from './system-agent.neo4j.repository';
 import { SystemAgentRepository } from './system-agent.repository';
@@ -49,6 +50,7 @@ import { UserService } from './user.service';
     },
     AddActorLabelMigration,
     AddUserNameLabelMigration,
+    DefaultUserStatusMigration,
   ],
   exports: [
     UserService,

test/user.e2e-spec.ts

@@ -2,6 +2,7 @@ import { faker } from '@faker-js/faker';
 import { times } from 'lodash';
 import { firstLettersOfWords, isValidId } from '~/common';
 import { graphql, type InputOf } from '~/graphql';
+import { UserStatus } from '../src/components/user/dto';
 import {
   createEducation,
   createOrganization,
@@ -87,7 +88,7 @@ describe('User e2e', () => {
     expect(actual.displayLastName.value).toBe(user.displayLastName);
     expect(actual.phone.value).toBeNull();
     expect(actual.about.value).toBeNull();
-    expect(actual.status.value).toBeNull();
+    expect(actual.status.value).toBe(UserStatus.Active);
     expect(actual.timezone.value?.name).toBe(user.timezone);
   });