add indexes and update updateUserInfo endpoint to accept new prop

Author: AmarTrebinjac

__tests__/boot.ts

@@ -159,6 +159,7 @@ const LOGGED_IN_BODY = {
     clickbaitTries: null,
     hasLocationSet: false,
     location: null,
+    hideExperience: false,
   },
   marketingCta: null,
   feeds: [],

__tests__/updateUserInfo.ts

@@ -96,6 +96,7 @@ describe('mutation updateUserInfo', () => {
           country
           city
         }
+        hideExperience
       }
     }
   `;
@@ -662,4 +663,53 @@ describe('mutation updateUserInfo', () => {
     expect(updatedUser?.readmeHtml).toContain('<h1>');
     // locationId check skipped - requires DatasetLocation records
   });
+
+  it('should update hideExperience', async () => {
+    loggedUser = '1';
+    const repo = con.getRepository(User);
+
+    // Verify initial state is false
+    const user = await repo.findOneBy({ id: loggedUser });
+    expect(user?.hideExperience).toBe(false);
+
+    const res = await client.mutate(MUTATION, {
+      variables: {
+        data: {
+          hideExperience: true,
+          username: 'testuser',
+          name: 'Test User',
+        },
+      },
+    });
+
+    expect(res.errors).toBeFalsy();
+    expect(res.data.updateUserInfo.hideExperience).toBe(true);
+
+    const updatedUser = await repo.findOneBy({ id: loggedUser });
+    expect(updatedUser?.hideExperience).toBe(true);
+  });
+
+  it('should update hideExperience to false', async () => {
+    loggedUser = '1';
+    const repo = con.getRepository(User);
+
+    // Set hideExperience to true first
+    await repo.update({ id: loggedUser }, { hideExperience: true });
+
+    const res = await client.mutate(MUTATION, {
+      variables: {
+        data: {
+          hideExperience: false,
+          username: 'testuser',
+          name: 'Test User',
+        },
+      },
+    });
+
+    expect(res.errors).toBeFalsy();
+    expect(res.data.updateUserInfo.hideExperience).toBe(false);
+
+    const updatedUser = await repo.findOneBy({ id: loggedUser });
+    expect(updatedUser?.hideExperience).toBe(false);
+  });
 });

src/entity/user/User.ts

@@ -345,6 +345,7 @@ export class User {
   @Column({ type: 'text', default: null })
   locationId: string | null;
 
+  @Index('IDX_user_hideExperience')
   @Column({ type: 'boolean', default: false })
   hideExperience: boolean;
 

src/migration/1764520729009-UserHideExperience.ts

@@ -1,13 +1,19 @@
 import { MigrationInterface, QueryRunner } from 'typeorm';
 
 export class UserHideExperience1764520729009 implements MigrationInterface {
+  name = 'UserHideExperience1764520729009';
+
   public async up(queryRunner: QueryRunner): Promise<void> {
     await queryRunner.query(
       `ALTER TABLE "user" ADD IF NOT EXISTS "hideExperience" boolean NOT NULL DEFAULT false`,
     );
+    await queryRunner.query(
+      `CREATE INDEX IF NOT EXISTS "IDX_user_hideExperience" ON "user" ("hideExperience")`,
+    );
   }
 
   public async down(queryRunner: QueryRunner): Promise<void> {
+    await queryRunner.query(`DROP INDEX IF EXISTS "IDX_user_hideExperience"`);
     await queryRunner.query(
       `ALTER TABLE "user" DROP COLUMN IF EXISTS "hideExperience"`,
     );

src/routes/boot.ts

@@ -479,6 +479,7 @@ const getUser = (
       'locationId',
       'readme',
       'language',
+      'hideExperience',
     ],
   });
 

src/schema/users.ts

@@ -205,6 +205,7 @@ export interface GQLUpdateUserInfoInput extends GQLUpdateUserInput {
   externalLocationId?: string;
   cover?: string;
   readme?: string;
+  hideExperience?: boolean;
 }
 
 interface GQLUserParameters {
@@ -534,6 +535,10 @@ export const typeDefs = /* GraphQL */ `
     Where the user is located
     """
     location: DatasetLocation
+    """
+    Whether to hide user's experience
+    """
+    hideExperience: Boolean
   }
 
   """
@@ -824,6 +829,10 @@ export const typeDefs = /* GraphQL */ `
     The user's readme
     """
     readme: String
+    """
+    Whether to hide user's experience
+    """
+    hideExperience: Boolean
   }
 
   type TagsReadingStatus {

src/workers/opportunity/parseCVProfile.ts

@@ -102,7 +102,10 @@ export const parseCVProfileWorker: TypedNotificationWorker<'api.v1.candidate-pre
         if (existingExperiencesCount === 0) {
           await con
             .getRepository(User)
-            .update({ id: userId }, { hideExperience: true });
+            .update(
+              { id: userId, hideExperience: false },
+              { hideExperience: true },
+            );
         }
 
         return [