From fcd0b23bb4b94f4302b6a68200d868dfb31c3164 Mon Sep 17 00:00:00 2001
From: kemboi590 <bkemboi590@gmail.com>
Date: Mon, 22 Jul 2024 23:26:57 +0300
Subject: [PATCH] can now change user roles

---
 src/features/users/usersAPI.ts                |  2 +-
 src/pages/dashboard/main/Account.tsx          | 38 ++++++++++++-------
 .../ManageVehicles/CreateVehicleSpecs.tsx     |  2 +-
 3 files changed, 27 insertions(+), 15 deletions(-)

diff --git a/src/features/users/usersAPI.ts b/src/features/users/usersAPI.ts
index a921217..6e94433 100644
--- a/src/features/users/usersAPI.ts
+++ b/src/features/users/usersAPI.ts
@@ -7,7 +7,7 @@ export interface TUser {
     contact_phone: string;
     address: string;
     image_url: string;
-    role: "user" | "admin" | "both" | "disabled" | null;
+    role: "user" | "admin";
     password: string;
 }
 
diff --git a/src/pages/dashboard/main/Account.tsx b/src/pages/dashboard/main/Account.tsx
index b698c29..875a085 100644
--- a/src/pages/dashboard/main/Account.tsx
+++ b/src/pages/dashboard/main/Account.tsx
@@ -4,7 +4,7 @@ import { Toaster, toast } from 'sonner';
 
 function Account() {
   const { data: usersData, isLoading: usersLoading, error: usersError, refetch: refetchUsers } = usersAPI.useGetUsersQuery();
-  const [isDisabling, setIsDisabling] = useState<number | null>(null);
+  const [isUpdatingRole, setIsUpdatingRole] = useState<number | null>(null);
   const [updateUser] = usersAPI.useUpdateUserMutation();
 
   const [filters, setFilters] = useState({
@@ -29,17 +29,17 @@ function Account() {
     });
   };
 
-  const handleDisableUser = async (userId: number) => {
-    setIsDisabling(userId);
+  const handleRoleChange = async (userId: number, newRole: "user" | "admin") => {
+    setIsUpdatingRole(userId);
     try {
-      await updateUser({ id: userId, role: 'disabled' });
-      toast.success('User disabled successfully');
+      await updateUser({ id: userId, role: newRole });
+      toast.success('User role updated successfully');
       refetchUsers();
     } catch (error) {
-      console.error('Error disabling user', error);
-      toast.error('Error disabling user');
+      console.error('Error updating user role', error);
+      toast.error('Error updating user role');
     } finally {
-      setIsDisabling(null);
+      setIsUpdatingRole(null);
     }
   };
 
@@ -116,6 +116,7 @@ function Account() {
               <th>Email</th>
               <th>Contact Phone</th>
               <th>Address</th>
+              <th>Role</th>
               <th>Action</th>
             </tr>
           </thead>
@@ -127,19 +128,30 @@ function Account() {
                 <td>{user.email}</td>
                 <td>{user.contact_phone}</td>
                 <td>{user.address}</td>
+                <td>
+                  <select
+                    className="select select-bordered"
+                    value={user.role}
+                    onChange={(e) => handleRoleChange(user.id, e.target.value as "user" | "admin")}
+                    disabled={isUpdatingRole === user.id}
+                  >
+                    <option value="user">User</option>
+                    <option value="admin">Admin</option>
+                  </select>
+                </td>
                 <td>
                   <button
                     className="btn bg-webcolor text-text-light hover:text-black border-none"
-                    onClick={() => handleDisableUser(user.id)}
-                    disabled={isDisabling === user.id}
+                    onClick={() => handleRoleChange(user.id, user.role === 'user' ? 'admin' : 'user')}
+                    disabled={isUpdatingRole === user.id}
                   >
-                    {isDisabling === user.id ? (
+                    {isUpdatingRole === user.id ? (
                       <div className='flex items-center'>
                         <span className="loading loading-spinner text-text-light"></span>
-                        <span> Disabling...</span>
+                        <span> Updating...</span>
                       </div>
                     ) : (
-                      "Disable"
+                      "Change Role"
                     )}
                   </button>
                 </td>
diff --git a/src/pages/dashboard/main/ManageVehicles/CreateVehicleSpecs.tsx b/src/pages/dashboard/main/ManageVehicles/CreateVehicleSpecs.tsx
index d1a3df3..9d7b583 100644
--- a/src/pages/dashboard/main/ManageVehicles/CreateVehicleSpecs.tsx
+++ b/src/pages/dashboard/main/ManageVehicles/CreateVehicleSpecs.tsx
@@ -191,7 +191,7 @@ const CreateVehicle = () => {
               <p className="text-red-500">{errors.features?.message}</p>
             </div>
 
-            {/* Image upload input */}
+            {/* Imag e upload input */}
             <div className="form-control">
               <input type="file" className="input input-bordered bg-slate-200" accept="image/*" name="vehicle_image" onChange={handleImageUpload} />
             </div>