Fix dashboard link and set up proper linking to avoid page reloads

inventory/src/App.tsx

@@ -51,36 +51,20 @@ function App() {
       <Toaster richColors position="top-center" />
       <Routes>
         <Route path="/login" element={<Login />} />
-        <Route
+        {isLoggedIn ? (
-          path="/"
+          <Route element={<MainLayout />}>
-          element={
+            <Route path="/" element={<Dashboard />} />
-            isLoggedIn ? (
+            <Route path="/products" element={<Products />} />
-              <Navigate to="/dashboard" replace />
+            <Route path="/import" element={<Import />} />
-            ) : (
+            <Route path="/orders" element={<Orders />} />
-              <Navigate to="/login" replace />
+            <Route path="/purchase-orders" element={<PurchaseOrders />} />
-            )
+            <Route path="/analytics" element={<Analytics />} />
-          }
+            <Route path="/settings" element={<Settings />} />
-        />
+            <Route path="*" element={<Navigate to="/" replace />} />
-        <Route
+          </Route>
-          path="*"
+        ) : (
-          element={
+          <Route path="*" element={<Navigate to="/login" replace />} />
-            isLoggedIn ? (
+        )}
-              <MainLayout>
-                <Routes>
-                  <Route path="/" element={<Dashboard />} />
-                  <Route path="/products" element={<Products />} />
-                  <Route path="/import" element={<Import />} />
-                  <Route path="/orders" element={<Orders />} />
-                  <Route path="/purchase-orders" element={<PurchaseOrders />} />
-                  <Route path="/analytics" element={<Analytics />} />
-                  <Route path="/settings" element={<Settings />} />
-                </Routes>
-              </MainLayout>
-            ) : (
-              <Navigate to="/login" replace />
-            )
-          }
-        />
       </Routes>
     </QueryClientProvider>
   );

inventory/src/components/layout/AppSidebar.tsx

@@ -20,7 +20,7 @@ import {
   SidebarMenuItem,
   SidebarSeparator,
 } from "@/components/ui/sidebar";
-import { useLocation, useNavigate } from "react-router-dom";
+import { useLocation, useNavigate, Link } from "react-router-dom";
 
 const items = [
   {
@@ -86,12 +86,12 @@ export function AppSidebar() {
                       tooltip={item.title}
                       isActive={isActive}
                     >
-                      <a href={item.url}>
+                      <Link to={item.url}>
                         <item.icon className="h-4 w-4" />
                         <span className="group-data-[collapsible=icon]:hidden">
                           {item.title}
                         </span>
-                      </a>
+                      </Link>
                     </SidebarMenuButton>
                   </SidebarMenuItem>
                 );
@@ -110,12 +110,12 @@ export function AppSidebar() {
                   tooltip="Settings"
                   isActive={location.pathname === "/settings"}
                 >
-                  <a href="/settings">
+                  <Link to="/settings">
                     <Settings className="h-4 w-4" />
                     <span className="group-data-[collapsible=icon]:hidden">
                       Settings
                     </span>
-                  </a>
+                  </Link>
                 </SidebarMenuButton>
               </SidebarMenuItem>
             </SidebarMenu>

inventory/src/components/layout/MainLayout.tsx

@@ -1,11 +1,8 @@
 import { SidebarProvider, SidebarTrigger } from "@/components/ui/sidebar";
 import { AppSidebar } from "./AppSidebar";
+import { Outlet } from "react-router-dom";
 
-interface MainLayoutProps {
+export function MainLayout() {
-  children: React.ReactNode;
-}
-
-export function MainLayout({ children }: MainLayoutProps) {
   return (
     <SidebarProvider defaultOpen>
       <div className="flex min-h-screen w-full pr-2">
@@ -13,9 +10,9 @@ export function MainLayout({ children }: MainLayoutProps) {
         <main className="flex-1 overflow-hidden">
           <div className="flex h-14 w-full items-center border-b px-4 gap-4">
             <SidebarTrigger />
-            </div>
+          </div>
           <div className="overflow-auto h-[calc(100vh-3.5rem)] max-w-[1500px]">
-            {children}
+            <Outlet />
           </div>
         </main>
       </div>