Merge pull request #16822 from kishanhitk/fix-auth-dashboard

fix: Supabase Studio giving success message even if the delete user operation fails

studio/components/interfaces/Auth/Users/UserDropdown.tsx

@@ -82,8 +82,7 @@ const UserDropdown = ({ user, canRemoveUser, canRemoveMFAFactors }: UserDropdown
           ui.setNotification({ category: 'success', message: `Successfully deleted ${user.email}` })
           PageState.users = PageState.users.filter((x: any) => x.id != user.id)
           PageState.totalUsers -= 1
-        } finally {
-        }
+        } catch (error) {}
       },
     })
   }
@@ -129,7 +128,7 @@ const UserDropdown = ({ user, canRemoveUser, canRemoveMFAFactors }: UserDropdown
           ) : null}
           <Dropdown.Separator />
           <Tooltip.Root delayDuration={0}>
-            <Tooltip.Trigger >
+            <Tooltip.Trigger>
               <Dropdown.Item
                 onClick={handleDeleteFactors}
                 icon={<IconShieldOff size="tiny" />}

studio/pages/api/auth/[ref]/users.ts

@@ -115,6 +115,12 @@ const handleDelete = async (req: NextApiRequest, res: NextApiResponse) => {
   const payload = req.body
   const query = { query: `DELETE from auth.users where id='${payload.id}';` }
 
-  const response = post(`${PG_META_URL}/query`, query, { headers })
+  const response = await post(`${PG_META_URL}/query`, query, { headers })
+
+  if (response.error) {
+    console.error('Delete Users POST:', response.error)
+    return res.status(400).json({ error: response.error })
+  }
+
   return res.status(200).json(response)
 }