fix public feature flag update (#10887)

## Context
upsert from typeorm does not seem to return keys that are not updated,
I'm reverting back to find/save since upsert is not consistent

packages/twenty-server/src/engine/core-modules/feature-flag/services/__tests__/feature-flag.service.spec.ts

@@ -25,6 +25,8 @@ describe('FeatureFlagService', () => {
     findOneBy: jest.fn(),
     find: jest.fn(),
     upsert: jest.fn(),
+    findOne: jest.fn(),
+    save: jest.fn(),
   };
 
   const workspaceId = 'workspace-id';
@@ -179,9 +181,7 @@ describe('FeatureFlagService', () => {
         workspaceId,
       };
 
-      mockFeatureFlagRepository.upsert.mockResolvedValue({
+      mockFeatureFlagRepository.save.mockResolvedValue(mockFeatureFlag);
-        generatedMaps: [mockFeatureFlag],
-      });
 
       (
         featureFlagValidator.assertIsFeatureFlagKey as jest.Mock
@@ -196,17 +196,11 @@ describe('FeatureFlagService', () => {
 
       // Assert
       expect(result).toEqual(mockFeatureFlag);
-      expect(mockFeatureFlagRepository.upsert).toHaveBeenCalledWith(
+      expect(mockFeatureFlagRepository.save).toHaveBeenCalledWith({
-        {
+        key: FeatureFlagKey[featureFlag],
-          key: FeatureFlagKey[featureFlag],
+        value,
-          value,
+        workspaceId,
-          workspaceId,
+      });
-        },
-        {
-          conflictPaths: ['workspaceId', 'key'],
-          skipUpdateIfNoValuesChanged: true,
-        },
-      );
     });
 
     it('should throw an exception when feature flag key is invalid', async () => {

packages/twenty-server/src/engine/core-modules/feature-flag/services/feature-flag.service.ts

@@ -82,16 +82,6 @@ export class FeatureFlagService {
     value: boolean;
     shouldBePublic?: boolean;
   }): Promise<FeatureFlag> {
-    if (shouldBePublic) {
-      publicFeatureFlagValidator.assertIsPublicFeatureFlag(
-        featureFlag,
-        new FeatureFlagException(
-          'Invalid feature flag key, flag is not public',
-          FeatureFlagExceptionCode.INVALID_FEATURE_FLAG_KEY,
-        ),
-      );
-    }
-
     featureFlagValidator.assertIsFeatureFlagKey(
       featureFlag,
       new FeatureFlagException(
@@ -100,18 +90,36 @@ export class FeatureFlagService {
       ),
     );
 
-    const upsertResult = await this.featureFlagRepository.upsert(
+    const featureFlagKey = FeatureFlagKey[featureFlag];
-      {
+
-        key: FeatureFlagKey[featureFlag],
+    if (shouldBePublic) {
-        value,
+      publicFeatureFlagValidator.assertIsPublicFeatureFlag(
+        featureFlagKey,
+        new FeatureFlagException(
+          'Invalid feature flag key, flag is not public',
+          FeatureFlagExceptionCode.INVALID_FEATURE_FLAG_KEY,
+        ),
+      );
+    }
+
+    const existingFeatureFlag = await this.featureFlagRepository.findOne({
+      where: {
+        key: featureFlagKey,
         workspaceId: workspaceId,
       },
-      {
+    });
-        conflictPaths: ['workspaceId', 'key'],
-        skipUpdateIfNoValuesChanged: true,
-      },
-    );
 
-    return upsertResult.generatedMaps[0] as FeatureFlag;
+    const featureFlagToSave = existingFeatureFlag
+      ? {
+          ...existingFeatureFlag,
+          value,
+        }
+      : {
+          key: featureFlagKey,
+          value,
+          workspaceId: workspaceId,
+        };
+
+    return await this.featureFlagRepository.save(featureFlagToSave);
   }
 }

packages/twenty-server/test/integration/graphql/suites/settings-permissions/workspace.integration-spec.ts

@@ -472,7 +472,7 @@ describe('workspace permissions', () => {
               `,
           variables: {
             input: {
-              publicFeatureFlag: 'TestFeature',
+              publicFeatureFlag: 'IsStripeIntegrationEnabled',
               value: true,
             },
           },