Merge pull request #358 from MerginMaps/fix-admin-deactivate-account

Fix: Deactivate account in admin
MerginMaps · Jan 16, 2025 · 3d6d3c3 · 3d6d3c3
2 parents ce515e3 + a7196ad
commit 3d6d3c3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/web-app/packages/admin-lib/src/modules/admin/types.ts b/web-app/packages/admin-lib/src/modules/admin/types.ts
@@ -19,8 +19,8 @@ export interface UsersParams extends PaginatedRequestParams {
 export type UsersResponse = PaginatedResponse<UserResponse>
 
 export interface UpdateUserData {
-  is_admin: boolean
-  active: boolean
+  is_admin?: boolean
+  active?: boolean
 }
 
 export interface CreateUserData {

diff --git a/web-app/packages/admin-lib/src/modules/admin/views/AccountDetailView.vue b/web-app/packages/admin-lib/src/modules/admin/views/AccountDetailView.vue
@@ -215,8 +215,7 @@ const changeStatusDialog = () => {
       await adminStore.updateUser({
         username: user.value.username,
         data: {
-          active: !user.value.active,
-          is_admin: user.value.is_admin
+          active: !user.value.active
         }
       })
     }
@@ -283,7 +282,6 @@ const switchAdminAccess = async () => {
       await adminStore.updateUser({
         username: user.value.username,
         data: {
-          active: user.value.active,
           is_admin: !user.value.is_admin
         }
       })