From eee288cd9181f6e6de6028d7d99d28085757dfe1 Mon Sep 17 00:00:00 2001 From: plubber <51789398+ericHgorski@users.noreply.github.com> Date: Wed, 20 Nov 2024 11:58:19 -0500 Subject: [PATCH] refactor: update setCookie function to accept hours instead of days --- src/hooks/useFeatureEnabled.ts | 4 +--- src/utils/cookies.ts | 6 +++--- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/src/hooks/useFeatureEnabled.ts b/src/hooks/useFeatureEnabled.ts index 2cd0acea..eb6c9f0b 100644 --- a/src/hooks/useFeatureEnabled.ts +++ b/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"); diff --git a/src/utils/cookies.ts b/src/utils/cookies.ts index 8e1f1e12..06fb2a02 100644 --- a/src/utils/cookies.ts +++ b/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=/`;