From 718986c416d2d3994b1b1228b49c9d1b9cf40471 Mon Sep 17 00:00:00 2001
From: William Grant <willmgrant@gmail.com>
Date: Mon, 3 Apr 2023 14:09:06 +0200
Subject: [PATCH] fix: improved feature release logic

---
 ts/util/releaseFeature.ts | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/ts/util/releaseFeature.ts b/ts/util/releaseFeature.ts
index f7fbb5d8d..a3b4e3ca0 100644
--- a/ts/util/releaseFeature.ts
+++ b/ts/util/releaseFeature.ts
@@ -1,8 +1,8 @@
 import { Data } from '../data/data';
 
 // TODO update to agreed value between platforms
-// const featureReleaseTimestamp = 1677488400000; // unix 27/02/2023 09:00
-const featureReleaseTimestamp = 1677661200000; // unix 01/03/2023 09:00
+const featureReleaseTimestamp = 1677488400000; // unix 27/02/2023 09:00
+// const featureReleaseTimestamp = 1677661200000; // unix 01/03/2023 09:00
 let isFeatureReleased: boolean | undefined;
 
 /**
@@ -35,7 +35,6 @@ export async function checkIsFeatureReleased(featureName: string): Promise<boole
     if (Date.now() >= featureReleaseTimestamp) {
       if (featureAlreadyReleased) {
         // Feature is already released and we don't need to update the db
-        window.log.info(`WIP: [releaseFeature]: ${featureName} is released`);
       } else {
         window.log.info(
           `WIP: [releaseFeature]: It is time to release ${featureName}. Releasing it now`
@@ -45,7 +44,7 @@ export async function checkIsFeatureReleased(featureName: string): Promise<boole
           value: true,
         });
       }
-      return true;
+      isFeatureReleased = true;
     } else {
       // Reset featureReleased to false if we have already released a feature since we have updated the featureReleaseTimestamp to a later date.
       // The alternative solution would be to do a db migration everytime we want to use this system.
@@ -54,10 +53,15 @@ export async function checkIsFeatureReleased(featureName: string): Promise<boole
           id: 'featureReleased',
           value: false,
         });
+        isFeatureReleased = false;
       }
     }
   }
 
-  window.log.info(`WIP: [releaseFeature]: ${featureName} has not been released yet`);
-  return false;
+  window.log.info(
+    `WIP: [releaseFeature]: ${featureName} ${
+      Boolean(isFeatureReleased) ? 'is released' : 'has not been released yet'
+    }`
+  );
+  return Boolean(isFeatureReleased);
 }