Merge pull request #421 from skip-mev/eg/dont-persist-feature

refactor: update setCookie function to accept hours instead of days

src/hooks/useFeatureEnabled.ts

@@ -14,9 +14,7 @@ export function useFeatureEnabled(featureName: string): boolean {
       if (featureEnabled === undefined) {
         const randomNumber = Math.random() * 100;
         featureEnabled = randomNumber < rolloutPercentage ? "true" : "false";
-
-        // Set the cookie for 1 day
-        setCookie(featureName, featureEnabled, 1);
+        setCookie(featureName, featureEnabled);
       }
 
       setIsFeatureEnabled(featureEnabled === "true");

src/utils/cookies.ts

@@ -9,11 +9,11 @@ export const getCookie = (name: string): string | undefined => {
   return undefined;
 };
 
-export const setCookie = (name: string, value: string, days?: number): void => {
+export const setCookie = (name: string, value: string, hours?: number): void => {
   let expires = "";
-  if (days) {
+  if (hours) {
     const date = new Date();
-    date.setTime(date.getTime() + days * 24 * 60 * 60 * 1000);
+    date.setTime(date.getTime() + hours * 60 * 60 * 1000);
     expires = `; expires=${date.toUTCString()}`;
   }
   document.cookie = `${name}=${encodeURIComponent(value || "")}${expires}; path=/`;