fix: resolve login crash, infinite renders, and logout scroll bugs (#1319)

Author: Aryan-Verma-999

packages/react/src/views/ChatBody/ChatBody.js

@@ -86,6 +86,11 @@ const ChatBody = ({
     (state) => state.isUserAuthenticated
   );
 
+  const isUserAuthenticatedRef = useRef(isUserAuthenticated);
+  useEffect(() => {
+    isUserAuthenticatedRef.current = isUserAuthenticated;
+  }, [isUserAuthenticated]);
+
   const username = useUserStore((state) => state.username);
 
   const { getMessagesAndRoles, fetchAndSetPermissions, permissionsRef } =
@@ -217,7 +222,8 @@ const ChatBody = ({
       if (
         messageListRef.current.scrollTop === 0 &&
         !loadingOlderMessages &&
-        hasMoreMessages
+        hasMoreMessages &&
+        (isUserAuthenticatedRef.current || anonymousMode)
       ) {
         setLoadingOlderMessages(true);
 

packages/react/src/views/EmbeddedChat.js

@@ -26,11 +26,13 @@ import GlobalStyles from './GlobalStyles';
 import { overrideECProps } from '../lib/overrideECProps';
 
 const EmbeddedChat = (props) => {
-  const [config, setConfig] = useState(() => props);
+  const [remoteOverrides, setRemoteOverrides] = useState({});
 
-  useEffect(() => {
-    setConfig(props);
-  }, [props]);
+  const config = useMemo(
+    () => ({ ...props, ...remoteOverrides }),
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [props, remoteOverrides]
+  );
 
   const {
     isClosable = false,
@@ -52,14 +54,18 @@ const EmbeddedChat = (props) => {
     className = '',
     style = {},
     hideHeader = false,
-    auth = {
-      flow: 'PASSWORD',
-    },
+    auth: authProp = null,
     secure = false,
     dark = false,
     remoteOpt = false,
   } = config;
 
+  const auth = useMemo(
+    () => authProp ?? { flow: 'PASSWORD' },
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [authProp?.flow, authProp?.credentials]
+  );
+
   const hasMounted = useRef(false);
   const { classNames, styleOverrides } = useComponentOverrides('EmbeddedChat');
   const [fullScreen, setFullScreen] = useState(false);
@@ -208,7 +214,7 @@ const EmbeddedChat = (props) => {
 
         if (appInfo) {
           const remoteConfig = appInfo.propConfig;
-          setConfig((prevConfig) => overrideECProps(prevConfig, remoteConfig));
+          setRemoteOverrides((prev) => overrideECProps(prev, remoteConfig));
         }
       } catch (error) {
         console.error('Error fetching remote config:', error?.message || error);
@@ -219,7 +225,7 @@ const EmbeddedChat = (props) => {
     if (remoteOpt) {
       getConfig();
     }
-  }, [RCInstance, remoteOpt, setConfig, setIsSynced]);
+  }, [RCInstance, remoteOpt, setIsSynced]);
 
   const ECOptions = useMemo(
     () => ({

packages/ui-elements/src/components/Modal/Modal.js

@@ -72,4 +72,4 @@ export const Modal = forwardRef(
   }
 );
 
-Modal.displayName = Modal;
+Modal.displayName = 'Modal';