Refactor users.js for minor updates and delete users.sql

Co-authored-by: cherhchen <[email protected]>
ctc-uci · Dec 13, 2023 · cabe258 · cabe258
1 parent 49e1bca
commit cabe258
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 33 deletions.
diff --git a/routes/users.js b/routes/users.js
@@ -32,7 +32,7 @@ userRouter.post('/', async (req, res) => {
       approved,
     ]);
     res.status(201).json({
-      status: 'Success',
+      id,
     });
   } catch (err) {
     res.status(500).json({
@@ -49,7 +49,7 @@ userRouter.put('/:uid', async (req, res) => {
       `UPDATE users SET approved = TRUE WHERE id = $1 RETURNING *;`,
       [uid],
     );
-    return res.status(200).send(updatedApproval[0]);
+    return res.status(200).send(keysToCamel(updatedApproval));
   } catch (err) {
     return res.status(500).send(err.message);
   }
@@ -58,8 +58,8 @@ userRouter.put('/:uid', async (req, res) => {
 userRouter.delete('/:uid', async (req, res) => {
   try {
     const { uid } = req.params;
-    await db.query(`DELETE FROM users WHERE id = $1;`, [uid]);
-    res.status(200).send('Deleted user');
+    const deletedUser = await db.query(`DELETE FROM users WHERE id = $1 RETURNING *;`, [uid]);
+    res.status(200).send(keysToCamel(deletedUser));
   } catch (err) {
     res.status(500).send(err.message);
   }

diff --git a/server/queries/users.sql b/server/queries/users.sql