From 96e072e02cdcb2db32ac8b7dd4566cc26906a4ed Mon Sep 17 00:00:00 2001
From: Charles Bochet <charles@twenty.com>
Date: Wed, 18 Dec 2024 17:15:00 +0100
Subject: [PATCH] Fix

---
 .../SignInAppNavigationDrawerMock.tsx         | 21 +++++++++----------
 .../components/NavigationDrawer.tsx           |  8 +++----
 2 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/packages/twenty-front/src/modules/sign-in-background-mock/components/SignInAppNavigationDrawerMock.tsx b/packages/twenty-front/src/modules/sign-in-background-mock/components/SignInAppNavigationDrawerMock.tsx
index d29cf365f8b0..383b4093e6a2 100644
--- a/packages/twenty-front/src/modules/sign-in-background-mock/components/SignInAppNavigationDrawerMock.tsx
+++ b/packages/twenty-front/src/modules/sign-in-background-mock/components/SignInAppNavigationDrawerMock.tsx
@@ -1,12 +1,11 @@
 import { SupportDropdown } from '@/support/components/SupportDropdown';
-import {
-  NavigationDrawer,
-  NavigationDrawerProps,
-} from '@/ui/navigation/navigation-drawer/components/NavigationDrawer';
+import { NavigationDrawer } from '@/ui/navigation/navigation-drawer/components/NavigationDrawer';
 
 import { NavigationDrawerSectionForObjectMetadataItems } from '@/object-metadata/components/NavigationDrawerSectionForObjectMetadataItems';
 import { NavigationDrawerItem } from '@/ui/navigation/navigation-drawer/components/NavigationDrawerItem';
 import { NavigationDrawerSection } from '@/ui/navigation/navigation-drawer/components/NavigationDrawerSection';
+import { DEFAULT_WORKSPACE_LOGO } from '@/ui/navigation/navigation-drawer/constants/DefaultWorkspaceLogo';
+import { DEFAULT_WORKSPACE_NAME } from '@/ui/navigation/navigation-drawer/constants/DefaultWorkspaceName';
 import styled from '@emotion/styled';
 import { IconSearch, IconSettings, useIsMobile } from 'twenty-ui';
 import { generatedMockObjectMetadataItems } from '~/testing/mock-data/generatedMockObjectMetadataItems';
@@ -62,14 +61,14 @@ export const SignInAppNavigationDrawerMock = ({
 
   const footer = <SupportDropdown />;
 
-  const drawerProps: NavigationDrawerProps = {
-    children,
-    footer,
-  };
-
   return (
-    <NavigationDrawer className={className} footer={drawerProps.footer}>
-      {drawerProps.children}
+    <NavigationDrawer
+      className={className}
+      footer={footer}
+      logo={DEFAULT_WORKSPACE_LOGO}
+      title={DEFAULT_WORKSPACE_NAME}
+    >
+      {children}
     </NavigationDrawer>
   );
 };
diff --git a/packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawer.tsx b/packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawer.tsx
index 3d8839b30601..27ad2f25a31f 100644
--- a/packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawer.tsx
+++ b/packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawer.tsx
@@ -18,8 +18,8 @@ export type NavigationDrawerProps = {
   children: ReactNode;
   className?: string;
   footer?: ReactNode;
-  logo?: string;
-  title?: string;
+  logo: string;
+  title: string;
 };
 
 const StyledAnimatedContainer = styled(motion.div)<{ isSettings?: boolean }>`
@@ -115,8 +115,8 @@ export const NavigationDrawer = ({
           !isMobile && <NavigationDrawerBackButton title={title} />
         ) : (
           <NavigationDrawerHeader
-            name={title ?? ''}
-            logo={logo ?? ''}
+            name={title}
+            logo={logo}
             showCollapseButton={isHovered}
           />
         )}