From 1a78682695708242e1f9dc2ef848ace4df083cc7 Mon Sep 17 00:00:00 2001
From: alphaeusmote <41258468-alphaeusmote@users.noreply.replit.com>
Date: Thu, 10 Apr 2025 00:35:25 +0000
Subject: [PATCH] Improve sidebar navigation by removing unnecessary nesting

Replit-Commit-Author: Agent
Replit-Commit-Session-Id: 9111ef36-26c8-4085-84ca-a35dc1fec1b5
Replit-Commit-Screenshot-Url: https://storage.googleapis.com/screenshot-production-us-central1/7083d608-d6d3-4a6a-9a27-6286c5109627/844ef557-cda0-4b3b-a7a3-e380ec24cde4.jpg
---
 client/src/components/layouts/sidebar.tsx | 56 +++++++++++------------
 server/database-storage.ts                |  2 +-
 2 files changed, 27 insertions(+), 31 deletions(-)

diff --git a/client/src/components/layouts/sidebar.tsx b/client/src/components/layouts/sidebar.tsx
index 7506449..7fad45d 100644
--- a/client/src/components/layouts/sidebar.tsx
+++ b/client/src/components/layouts/sidebar.tsx
@@ -114,21 +114,19 @@ export function Sidebar() {
 
       <nav className="flex-1 py-4 px-2 space-y-1 overflow-y-auto">
         {navItems.map((item) => (
-          <div key={item.href}>
-            <Link href={item.href}>
-              <div
-                className={cn(
-                  "flex items-center px-3 py-2 text-sm font-medium rounded-md cursor-pointer",
-                  location === item.href
-                    ? "bg-primary text-primary-foreground"
-                    : "hover:bg-accent hover:text-accent-foreground"
-                )}
-              >
-                {item.icon}
-                {item.title}
-              </div>
-            </Link>
-          </div>
+          <Link key={item.href} href={item.href}>
+            <div
+              className={cn(
+                "flex items-center px-3 py-2 text-sm font-medium rounded-md cursor-pointer",
+                location === item.href
+                  ? "bg-primary text-primary-foreground"
+                  : "hover:bg-accent hover:text-accent-foreground"
+              )}
+            >
+              {item.icon}
+              {item.title}
+            </div>
+          </Link>
         ))}
         
         {filteredAdminItems.length > 0 && (
@@ -137,21 +135,19 @@ export function Sidebar() {
               Administration
             </div>
             {filteredAdminItems.map((item) => (
-              <div key={item.href}>
-                <Link href={item.href}>
-                  <div
-                    className={cn(
-                      "mt-1 flex items-center px-3 py-2 text-sm font-medium rounded-md cursor-pointer",
-                      location === item.href
-                        ? "bg-primary text-primary-foreground"
-                        : "hover:bg-accent hover:text-accent-foreground"
-                    )}
-                  >
-                    {item.icon}
-                    {item.title}
-                  </div>
-                </Link>
-              </div>
+              <Link key={item.href} href={item.href}>
+                <div
+                  className={cn(
+                    "mt-1 flex items-center px-3 py-2 text-sm font-medium rounded-md cursor-pointer",
+                    location === item.href
+                      ? "bg-primary text-primary-foreground"
+                      : "hover:bg-accent hover:text-accent-foreground"
+                  )}
+                >
+                  {item.icon}
+                  {item.title}
+                </div>
+              </Link>
             ))}
           </div>
         )}
diff --git a/server/database-storage.ts b/server/database-storage.ts
index 8352ba4..fe2854e 100644
--- a/server/database-storage.ts
+++ b/server/database-storage.ts
@@ -226,7 +226,7 @@ export class DatabaseStorage implements IStorage {
     userId?: string
   ): Promise<DnsHistory> {
     const [historyEntry] = await db.insert(dnsHistory).values({
-      record_id: recordId,
+      recordId,
       action,
       previousValue,
       newValue,