fix: guard against fetching appsettings many times (#3095)

Author: isekovanic

package/src/components/Chat/hooks/useAppSettings.ts

@@ -1,4 +1,4 @@
-import { useEffect, useState } from 'react';
+import { useEffect, useRef, useState } from 'react';
 
 import type { AppSettingsAPIResponse, StreamChat } from 'stream-chat';
 
@@ -12,9 +12,18 @@ export const useAppSettings = (
   initialisedDatabase: boolean,
 ): AppSettingsAPIResponse | null => {
   const [appSettings, setAppSettings] = useState<AppSettingsAPIResponse | null>(null);
+  const appSettingsPromise = useRef<Promise<AppSettingsAPIResponse | null>>(null);
+  const fetchedAppSettings = useRef(false);
   const isMounted = useIsMountedRef();
 
   useEffect(() => {
+    const fetchAppSettings = () => {
+      if (appSettingsPromise.current) {
+        return appSettingsPromise.current;
+      }
+      appSettingsPromise.current = client.getAppSettings();
+      return appSettingsPromise.current;
+    };
     /**
      * Fetches app settings from the backend when offline support is disabled.
      */
@@ -24,9 +33,10 @@ export const useAppSettings = (
       }
 
       try {
-        const appSettings = await client.getAppSettings();
+        const appSettings = await fetchAppSettings();
         if (isMounted.current) {
           setAppSettings(appSettings);
+          fetchedAppSettings.current = true;
         }
       } catch (error: unknown) {
         if (error instanceof Error) {
@@ -56,9 +66,10 @@ export const useAppSettings = (
       }
 
       try {
-        const appSettings = await client.getAppSettings();
-        if (isMounted.current) {
+        const appSettings = await fetchAppSettings();
+        if (isMounted.current && appSettings) {
           setAppSettings(appSettings);
+          fetchedAppSettings.current = true;
           await dbApi.upsertAppSettings({
             appSettings,
             currentUserId: client.userID as string,
@@ -72,6 +83,10 @@ export const useAppSettings = (
     };
 
     async function enforeAppSettings() {
+      if (fetchedAppSettings.current) {
+        return;
+      }
+
       if (enableOfflineSupport) {
         await enforceAppSettingsWithOfflineSupport();
       } else {