From 081715e2648bc9f5264a6998d79fb6839a60e22c Mon Sep 17 00:00:00 2001
From: Rajat Saxena <hi@rajatsaxena.dev>
Date: Mon, 19 Feb 2024 19:06:31 +0530
Subject: [PATCH] Bug fix: apps with empty names are not updateable

---
 apps/web/app/app/[keyid]/settings/actions.tsx             | 6 +++---
 apps/web/app/app/[keyid]/settings/page.tsx                | 2 +-
 .../web/app/app/[keyid]/settings/update-settings-form.tsx | 8 ++++----
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/apps/web/app/app/[keyid]/settings/actions.tsx b/apps/web/app/app/[keyid]/settings/actions.tsx
index 83dfef29..03b095f9 100644
--- a/apps/web/app/app/[keyid]/settings/actions.tsx
+++ b/apps/web/app/app/[keyid]/settings/actions.tsx
@@ -12,11 +12,11 @@ export async function updateAppName(
     formData: FormData
 ) {
     const newName = formData.get("newName") as string;
-    const name = formData.get("name") as string;
+    const keyId = formData.get("keyId") as string;
     if (!newName) {
         return { success: false, error: "Name is required" };
     }
-    if (!name) {
+    if (!keyId) {
         return { success: false, error: "Bad request" };
     }
 
@@ -36,7 +36,7 @@ export async function updateAppName(
         await ApikeyModel.updateOne(
             {
                 userId: dbUser._id,
-                name,
+                keyId,
             },
             { $set: { name: newName } }
         );
diff --git a/apps/web/app/app/[keyid]/settings/page.tsx b/apps/web/app/app/[keyid]/settings/page.tsx
index 4a69320b..e61cdc2e 100644
--- a/apps/web/app/app/[keyid]/settings/page.tsx
+++ b/apps/web/app/app/[keyid]/settings/page.tsx
@@ -54,7 +54,7 @@ export default async function Settings({
                     <CopyApikeyButton apikey={apikey.key} />
                 </div>
             </div>
-            <UpdateSettingsForm name={apikey.name} />
+            <UpdateSettingsForm keyId={apikey.keyId} />
             <Separator className="my-8" />
             <DeleteAppButton apikey={apikey} />
         </section>
diff --git a/apps/web/app/app/[keyid]/settings/update-settings-form.tsx b/apps/web/app/app/[keyid]/settings/update-settings-form.tsx
index 45650fb5..319d8502 100644
--- a/apps/web/app/app/[keyid]/settings/update-settings-form.tsx
+++ b/apps/web/app/app/[keyid]/settings/update-settings-form.tsx
@@ -9,11 +9,11 @@ import { useEffect, useState } from "react";
 import { useRouter } from "next/navigation";
 import { useToast } from "@/components/ui/use-toast";
 
-export default function UpdateSettingsForm({ name }: { name: string }) {
+export default function UpdateSettingsForm({ keyId }: { keyId: string }) {
     const [state, updateNameAction] = useFormState(updateAppName, {
         success: false,
     });
-    const [newName, setNewName] = useState(name);
+    const [newName, setNewName] = useState("");
     const router = useRouter();
     const { toast } = useToast();
 
@@ -38,7 +38,7 @@ export default function UpdateSettingsForm({ name }: { name: string }) {
 
     return (
         <div>
-            <Label htmlFor="name" className="mb-2">
+            <Label htmlFor="newName" className="mb-2">
                 App name
             </Label>
             <form action={updateNameAction} className="flex gap-2">
@@ -47,7 +47,7 @@ export default function UpdateSettingsForm({ name }: { name: string }) {
                     value={newName}
                     onChange={(e) => setNewName(e.target.value)}
                 />
-                <Input name="name" value={name} type="hidden" />
+                <Input name="keyId" value={keyId} type="hidden" />
                 <Submit>Save</Submit>
             </form>
         </div>