From a4463468a186704e0741ff6a061e51a1d114a7f3 Mon Sep 17 00:00:00 2001
From: Antony David <antonydavid945@gmail.com>
Date: Sat, 20 Jul 2024 15:00:15 +0200
Subject: [PATCH] fix(admin): addresses form issues (#275)

---
 .../admin/app/ui/dashboard/addresses/AddAddress.tsx | 10 +++++++---
 .../admin/app/ui/dashboard/addresses/AddressRow.tsx |  6 +++++-
 apps/admin/app/ui/dashboard/addresses/EditForm.tsx  | 13 +++++++++----
 3 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/apps/admin/app/ui/dashboard/addresses/AddAddress.tsx b/apps/admin/app/ui/dashboard/addresses/AddAddress.tsx
index a6fb74e9..150844c8 100644
--- a/apps/admin/app/ui/dashboard/addresses/AddAddress.tsx
+++ b/apps/admin/app/ui/dashboard/addresses/AddAddress.tsx
@@ -154,7 +154,7 @@ function addAddress({ addresses, setAddresses }: Props): JSX.Element {
                         <FormItem>
                           <Label className="font-bold">Numéro</Label>
                           <FormControl>
-                            <Input {...field} type="number" />
+                            <Input {...field} type="number" min={1} />
                           </FormControl>
                           <FormMessage />
                         </FormItem>
@@ -214,7 +214,7 @@ function addAddress({ addresses, setAddresses }: Props): JSX.Element {
                         <FormItem>
                           <Label className="font-bold">Code postal</Label>
                           <FormControl>
-                            <Input {...field} />
+                            <Input {...field} type="number" min={0} />
                           </FormControl>
                           <FormMessage />
                         </FormItem>
@@ -223,7 +223,11 @@ function addAddress({ addresses, setAddresses }: Props): JSX.Element {
                   </div>
                 </div>
                 <div className="flex gap-4 mt-4">
-                  <Button type="submit" className="w-full">
+                  <Button
+                    type="submit"
+                    className="w-full"
+                    disabled={form.formState.isSubmitting || !form.formState.isValid}
+                  >
                     Créer
                   </Button>
                   <Button variant="secondary" type="button" onClick={() => setOpen(false)} className="w-full">
diff --git a/apps/admin/app/ui/dashboard/addresses/AddressRow.tsx b/apps/admin/app/ui/dashboard/addresses/AddressRow.tsx
index 518b9b96..91bd72c5 100644
--- a/apps/admin/app/ui/dashboard/addresses/AddressRow.tsx
+++ b/apps/admin/app/ui/dashboard/addresses/AddressRow.tsx
@@ -79,6 +79,10 @@ function AddressRow(address: AddressProps) {
     setOpenDelete(false);
   }
 
+  if (!address || name === 'Supprimé') {
+    return null;
+  }
+
   return (
     <TableRow key={address.id}>
       <TableCell className="font-medium">{name}</TableCell>
@@ -131,7 +135,7 @@ function AddressRow(address: AddressProps) {
                       <DialogDescription>
                         <div className="mb-4">Êtes-vous sûr de vouloir supprimer cet adresse ?</div>
                         <div className="flex w-full justify-end gap-4">
-                          <Button variant="destructive" onClick={deleteAddress}>
+                          <Button variant="destructive" onClick={deleteAddress} disabled={name === 'Supprimé'}>
                             Supprimer
                           </Button>
                           <Button variant="secondary" onClick={() => setOpenDelete(false)}>
diff --git a/apps/admin/app/ui/dashboard/addresses/EditForm.tsx b/apps/admin/app/ui/dashboard/addresses/EditForm.tsx
index 6509df8d..bc363655 100644
--- a/apps/admin/app/ui/dashboard/addresses/EditForm.tsx
+++ b/apps/admin/app/ui/dashboard/addresses/EditForm.tsx
@@ -39,7 +39,10 @@ function EditForm(props: EditFormProps): JSX.Element {
       .min(5, { message: 'Le code postal doit contenir 5 chiffres' }),
     complement: z.string().optional(),
     city: z.string().min(1, { message: 'Le champ est requis' }),
-    number: z.number({ message: 'Le numéro doit être un nombre' }).int().min(1, { message: 'Le champ est requis' }),
+    number: z.coerce
+      .number({ message: 'Le numéro doit être un nombre' })
+      .int()
+      .min(1, { message: 'Le champ est requis' }),
     name: z.string().optional(),
   });
 
@@ -124,7 +127,7 @@ function EditForm(props: EditFormProps): JSX.Element {
                 <FormItem>
                   <Label className="font-bold">Numéro</Label>
                   <FormControl>
-                    <Input {...field} type="number" />
+                    <Input {...field} type="number" min={1} />
                   </FormControl>
                   <FormMessage />
                 </FormItem>
@@ -184,7 +187,7 @@ function EditForm(props: EditFormProps): JSX.Element {
                 <FormItem>
                   <Label className="font-bold">Code postal</Label>
                   <FormControl>
-                    <Input {...field} />
+                    <Input {...field} type="number" min={1} />
                   </FormControl>
                   <FormMessage />
                 </FormItem>
@@ -193,7 +196,9 @@ function EditForm(props: EditFormProps): JSX.Element {
           </div>
         </div>
         <div className="flex gap-4 mt-4">
-          <Button type="submit">Modifier</Button>
+          <Button type="submit" disabled={form.formState.isSubmitting || !form.formState.isValid}>
+            Modifier
+          </Button>
           <Button
             variant="secondary"
             onClick={(e) => {