164798024-feature(notification): update notification

- edit variable names
andela · Apr 24, 2019 · 98ed462 · 98ed462
1 parent 8e091d4
commit 98ed462
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/server/controllers/notification.controllers.js b/server/controllers/notification.controllers.js
@@ -44,21 +44,21 @@ const updateIsSeenColumn = async (req, res) => {
  * @param {*} res
  * @returns {object} All the notification belonging to the user
  */
-const getAllNotification = async (req, res) => {
+const getAllNotifications = async (req, res) => {
   try {
-    const notification = await Notification.findAndCountAll({
+    const notifications = await Notification.findAndCountAll({
       where: {
         user_id: req.user.userObj.id,
         is_seen: false,
       },
     });
-    if (!notification || notification.count === 0) {
+    if (!notifications || notifications.count === 0) {
       return res.status(200).json({
         message: "You don't have any notifications",
       });
     }
     return res.status(200).json({
-      data: notification,
+      data: notifications,
     });
   } catch (err) {
     return serverError;
@@ -67,7 +67,7 @@ const getAllNotification = async (req, res) => {
 
 const notificationController = {
   updateIsSeenColumn,
-  getAllNotification,
+  getAllNotifications,
 };
 
 export default notificationController;
diff --git a/server/routes/notification.routes.js b/server/routes/notification.routes.js
@@ -67,7 +67,7 @@ router.get(
 router.get(
   '/notifications',
   tokenValidator.verifyToken,
-  updateIsSeenColumn.getAllNotification
+  updateIsSeenColumn.getAllNotifications
 );
 
 export default router;