From 2fba7c82d7bba816bcf0d62951b04f4e875b5996 Mon Sep 17 00:00:00 2001
From: joshuasilva414 <joshuasilva414@gmail.com>
Date: Thu, 2 Jan 2025 21:19:53 -0600
Subject: [PATCH] fix  setItem action

---
 apps/web/src/actions/admin/modify-nav-item.ts   |  4 ++--
 .../admin/toggles/NavItemsManager.tsx           | 17 ++++++++++-------
 apps/web/src/lib/utils/server/redis.ts          |  3 ++-
 package.json                                    |  2 +-
 4 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/apps/web/src/actions/admin/modify-nav-item.ts b/apps/web/src/actions/admin/modify-nav-item.ts
index 99185e69..021268c0 100644
--- a/apps/web/src/actions/admin/modify-nav-item.ts
+++ b/apps/web/src/actions/admin/modify-nav-item.ts
@@ -21,7 +21,7 @@ export const setItem = adminAction
 			encodeURIComponent(name),
 		);
 		await hset(
-			"config:navitems:${encodeURIComponent(name)}",
+			`config:navitems:${encodeURIComponent(name)}`,
 			{
 				url,
 				name,
@@ -49,7 +49,7 @@ export const toggleItem = adminAction
 			ctx: { user, userId },
 		}) => {
 			await hset(
-				"config:navitems:${encodeURIComponent(name)}",
+				`config:navitems:${encodeURIComponent(name)}`,
 				{
 					enabled: statusToSet,
 				},
diff --git a/apps/web/src/components/admin/toggles/NavItemsManager.tsx b/apps/web/src/components/admin/toggles/NavItemsManager.tsx
index a24784c3..b81369af 100644
--- a/apps/web/src/components/admin/toggles/NavItemsManager.tsx
+++ b/apps/web/src/components/admin/toggles/NavItemsManager.tsx
@@ -60,13 +60,16 @@ export function NavItemsManager({ navItems }: NavItemsManagerProps) {
 				</TableHeader>
 				<TableBody>
 					{navItems.map((item) => (
-						<TableRow key={item.name}>
+						<TableRow key={item?.name ?? ""}>
 							<TableCell className="font-medium">
-								{item.name}
+								{item?.name ?? ""}
 							</TableCell>
 							<TableCell>
-								<Link className="underline" href={item.url}>
-									{item.url}
+								<Link
+									className="underline"
+									href={item?.url ?? ""}
+								>
+									{item?.url ?? ""}
 								</Link>
 							</TableCell>
 							<TableCell>
@@ -75,8 +78,8 @@ export function NavItemsManager({ navItems }: NavItemsManagerProps) {
 									onCheckedChange={(checked) => didToggle(item.name, checked)}
 								/> */}
 								<ToggleSwitch
-									itemStatus={item.enabled}
-									name={item.name}
+									itemStatus={item?.enabled ?? ""}
+									name={item?.name ?? ""}
 								/>
 							</TableCell>
 							<TableCell className="space-x-2 text-right">
@@ -85,7 +88,7 @@ export function NavItemsManager({ navItems }: NavItemsManagerProps) {
 								</Button>
 								<Button
 									onClick={() => {
-										execute(item.name);
+										execute(item?.name ?? "");
 									}}
 								>
 									Delete
diff --git a/apps/web/src/lib/utils/server/redis.ts b/apps/web/src/lib/utils/server/redis.ts
index 3339c109..efe25b87 100644
--- a/apps/web/src/lib/utils/server/redis.ts
+++ b/apps/web/src/lib/utils/server/redis.ts
@@ -53,6 +53,7 @@ export async function getAllNavItems() {
 }
 
 export function removeNavItem(name: string) {
+	console.log("Removing: ", includeEnvPrefix("config:navitemslist"));
 	const pipe = kv.pipeline();
 	pipe.srem(
 		includeEnvPrefix("config:navitemslist"),
@@ -69,7 +70,7 @@ export function parseRedisBoolean(
 	if (typeof value === "string") {
 		if (value === "true") return true;
 		if (value === "false") return false;
-	}
+	} 
 	if (typeof value === "boolean") return value;
 	return defaultValue !== undefined ? defaultValue : false;
 }
diff --git a/package.json b/package.json
index ed954ddf..e0144114 100644
--- a/package.json
+++ b/package.json
@@ -21,7 +21,7 @@
     "prettier": "^3.3.3",
     "turbo": "^2.0.9"
   },
-  "packageManager": "pnpm@8.3.1",
+  "packageManager": "pnpm@8.3.1+sha512.d264f6640bf4f09de7cfcc547568515bcf0613cf485a03e8ff16616fa69c4172b6f9a0a2925ee44fb060df565c9c9a8eaf061749e77af318cb77f6684a7051f3",
   "name": "hackkit",
   "dependencies": {
     "prettier-plugin-tailwindcss": "^0.6.5"