Merge pull request #6 from mrlazyg/dev

user route
noorsalimx · Mar 7, 2024 · 4ebf0ad · 4ebf0ad
2 parents 132903a + ed717b2
commit 4ebf0ad
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/controllers/UserController.js b/controllers/UserController.js
@@ -9,8 +9,17 @@ const getAllUsers = (req, res) => {
 };
 
 const getUser = (req, res) => {
+  const { userId } = req.params;
+  const user = users.find((id) => userId === id.email || userId === id._id);
+  if (!user) {
+    return res.status(404).send({
+      status: 'error',
+      message: 'user not found',
+    });
+  }
   res.send({
     status: 'success',
+    user: user || 'User not found',
   });
 };
 

diff --git a/routes/UserRoutes.js b/routes/UserRoutes.js
@@ -8,7 +8,7 @@ router
   .get(UserController.getAllUsers)
   .post(UserController.createUser);
 router
-  .route('/:id')
+  .route('/:userId')
   .get(UserController.getUser)
   .patch(UserController.updateUser)
   .delete(UserController.deleteUser);