fix: resolves build error due to no REDIS_URL defined for storybook env

- adds STORYBOOK_IS_STORYBOOK env var to prevent REDIS usage in storybook
Plant-for-the-Planet-org · Apr 11, 2024 · 9ea7417 · 9ea7417
1 parent 4895e73
commit 9ea7417
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 4 deletions.
diff --git a/.storybook/main.js b/.storybook/main.js
@@ -34,4 +34,9 @@ module.exports = {
   docs: {
     autodocs: true,
   },
+
+  env: (config) => ({
+    ...config,
+    STORYBOOK_IS_STORYBOOK: true,
+  }),
 };
diff --git a/src/redis-client.ts b/src/redis-client.ts
@@ -1,6 +1,10 @@
 import { createClient } from '@vercel/kv';
 
 const getRedisClient = () => {
+  if (process.env.STORYBOOK_IS_STORYBOOK) {
+    return null;
+  }
+
   if (!process.env.REDIS_URL) {
     throw new Error('REDIS_URL is not defined');
   }

diff --git a/src/theme/theme.ts b/src/theme/theme.ts
@@ -19,6 +19,9 @@ const getGlobalStyles = async () => {
   const fetchConfig = async () => {
     try {
       let tenantConfig;
+      if (process.env.STORYBOOK_IS_STORYBOOK) {
+        return null;
+      }
       if (typeof window !== 'undefined') {
         // Check if tenantConfig is stored in local storage
         const storedConfig = localStorage.getItem('tenantConfig');

diff --git a/src/utils/multiTenancy/helpers.ts b/src/utils/multiTenancy/helpers.ts
@@ -57,7 +57,8 @@ export const getTenantConfig = async (slug: string): Promise<Tenant> => {
       'env_missing';
     const caching_key = `${cachingKeyPrefix}_TENANT_CONFIG_${slug}`;
 
-    const tenant = await redisClient.get<Tenant>(caching_key);
+    const tenant =
+      redisClient !== null ? await redisClient.get<Tenant>(caching_key) : null;
 
     if (tenant) {
       return tenant;
@@ -75,9 +76,10 @@ export const getTenantConfig = async (slug: string): Promise<Tenant> => {
 
     const tenantConf = _tenantConf ?? defaultTenantConfig;
 
-    await redisClient.set(caching_key, JSON.stringify(tenantConf), {
-      ex: FIVE_HOURS,
-    });
+    redisClient !== null &&
+      (await redisClient.set(caching_key, JSON.stringify(tenantConf), {
+        ex: FIVE_HOURS,
+      }));
 
     return tenantConf as Tenant; // Ensure that the returned value is of type Tenant
   } catch (err) {