fix: add null check for the user (#474)

* fix: add null check for the user * refactor: uplift condition statement
datavisyn · Aug 16, 2024 · ed690a7 · ed690a7
1 parent 18ebcd2
commit ed690a7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/app/header/UserMenu.tsx b/src/app/header/UserMenu.tsx
@@ -20,6 +20,11 @@ export function UserMenu({ menu, user, color }: { menu: JSX.Element; user: strin
           size={28}
           variant="white"
         >
+          {/*
+            // NOTE: @dv-usama-ansari: We would not need this fix when we upgrade to Mantine v7.11
+            //  Extracting initials is a built-in feature in Mantine v7.11: https://mantine.dev/changelog/7-11-0/#avatar-initials
+            */}
+
           {user
             .split(' ')
             .map((name) => name[0])

diff --git a/src/app/header/VisynHeader.tsx b/src/app/header/VisynHeader.tsx
@@ -95,7 +95,7 @@ export function VisynHeader({
         {components?.logo === undefined ? <DatavisynLogo color={backgroundColor === 'white' ? 'black' : 'white'} /> : components?.logo}
         <Group gap={5}>
           {components?.userAvatar === undefined ? (
-            user ? (
+            user && user.name ? (
               <UserMenu menu={components?.userMenu} user={user.name} color={backgroundColor} />
             ) : null
           ) : (