fix(navbar): update TopNav component to use isLoggedIn for rendering links

rootcode-creator · rootcode-creator · commit 15145872bcd3 · 2026-03-28T04:19:56.000+06:00
diff --git a/src/app/actions/userActions.ts b/src/app/actions/userActions.ts
@@ -117,6 +117,6 @@ export async function getUserInfoForNav() {
         })
     } catch (error) {
         console.log(error);
-        throw error;
+        return null;
     }
 }
diff --git a/src/app/layout.tsx b/src/app/layout.tsx
@@ -21,6 +21,7 @@ export default async function RootLayout({
 }>) {
   const session = await auth();
   const userId = session?.user?.id || null;
+  const isLoggedIn = Boolean(userId);
 
   // Don't rely on session.user.image here: with JWT sessions it won't change
   // until the token is refreshed (often only on re-login).
@@ -33,7 +34,7 @@ export default async function RootLayout({
     <html lang="en">
       <body>
         <Providers profileComplete={profileComplete} userId={userId}>
-          <TopNav userInfo={userInfo} isAdmin={isAdmin} />
+          <TopNav userInfo={userInfo} isLoggedIn={isLoggedIn} isAdmin={isAdmin} />
           <main className="container mx-auto">
             {children}
           </main>
diff --git a/src/components/navbar/TopNav.tsx b/src/components/navbar/TopNav.tsx
@@ -18,11 +18,13 @@ type Props = {
     name: string | null;
     image: string | null;
   } | null;
+  isLoggedIn: boolean;
   isAdmin: boolean;
 };
 
 export default function TopNav({
   userInfo,
+  isLoggedIn,
   isAdmin,
 }: Props) {
 
@@ -71,7 +73,7 @@ export default function TopNav({
           </div>
         </NavbarBrand>
         <NavbarContent justify="center">
-          {userInfo &&
+          {isLoggedIn &&
             links.map((item) => (
               <NavLink
                 key={item.href}
@@ -84,7 +86,7 @@ export default function TopNav({
           justify="end"
           className="flex-1"
         >
-          {userInfo ? (
+          {isLoggedIn ? (
             <div className="ml-auto">
               <UserMenu userInfo={userInfo} />
             </div>

Original file line number	Diff line number	Diff line change
`@@ -117,6 +117,6 @@ export async function getUserInfoForNav() {`
`117`	`117`	`})`
`118`	`118`	`} catch (error) {`
`119`	`119`	`console.log(error);`
`120`		`- throw error;`
	`120`	`+ return null;`
`121`	`121`	`}`
`122`	`122`	`}`