bug(userId):return userid after updating user profile

To add userId to the attributes being returned
[#168203691]

src/controllers/auth.controllers.js

@@ -163,6 +163,7 @@ export default {
 
       return Helper.successResponse(response, 200, {
         firstName: updatedUser.dataValues.firstName,
+        id: updatedUser.dataValues.id,
         lastName: updatedUser.dataValues.lastName,
         bio: updatedUser.dataValues.bio,
         userName: updatedUser.dataValues.userName,

src/controllers/notification.controller.js

@@ -1,4 +1,7 @@
-import { fetchNotificationService } from '../services/notification.service';
+import {
+  fetchNotificationService,
+  readNotificationService
+} from '../services/notification.service';
 import Helper from '../services/helper';
 
 export default {
@@ -17,5 +20,22 @@ export default {
     } catch (error) {
       return error;
     }
+  },
+
+  /**
+     * @method readNotification 
+       read a notification
+     * 
+     * @param {*} request
+     * @param {*} response
+     */
+
+  async readNotification(request, response) {
+    try {
+      const value = await readNotificationService(request.params);
+      return Helper.successResponse(response, 200, value);
+    } catch (error) {
+      return error;
+    }
   }
 };

src/routes/v1/notification.route.js

@@ -3,7 +3,8 @@ import notificationController from '../../controllers/notification.controller';
 import authorization from '../../middlewares/auth.middleware';
 
 const { verifyToken } = authorization;
-const { fetchNotification } = notificationController;
+const { fetchNotification, readNotification } = notificationController;
 const router = express.Router();
 router.get('/', verifyToken, fetchNotification);
+router.put('/:notificationId', verifyToken, readNotification);
 export default router;

src/services/article.service.js

@@ -118,7 +118,7 @@ export const getArticleService = async data => {
       {
         model: User,
         as: 'author',
-        attributes: ['id', 'firstName', 'lastName', 'image']
+        attributes: ['id', 'firstName', 'lastName', 'image', 'userName', 'bio']
       },
       {
         model: Tag,

src/services/auth.service.js

@@ -53,7 +53,11 @@ export const signUpService = async body => {
     lastName: result.lastName,
     email: result.email,
     image: result.image,
-    token: getToken(result)
+    token: getToken(result),
+    userName: result.userName,
+    bio: result.bio,
+    id: result.id,
+    isNotified: result.isNotified
   };
 
   sendWelcomeEmail(

src/services/bookmark.service.js

@@ -62,6 +62,7 @@ export const getBookmarks = async userId => {
           'body',
           'image',
           'likesCount',
+          'isPublished',
           'viewsCount',
           'description'
         ],
@@ -79,6 +80,14 @@ export const getBookmarks = async userId => {
   if (userBookmarks.bookmarks.length < 1) {
     return { message: `You currently don't have any bookmarks` };
   }
+
+  if (userBookmarks.bookmarks) {
+    userBookmarks.bookmarks.forEach((article, index) => {
+      if (article.isPublished === false) {
+        userBookmarks.bookmarks.splice(index);
+      }
+    });
+  }
   return userBookmarks;
 };
 

src/services/notification.service.js

@@ -259,3 +259,26 @@ export const sendNotificationOnArticlePublish = async details => {
     return response;
   }
 };
+
+/**
+ * @method readNotificationService
+ * @description read a notification
+ *
+ * @param {Object} details details of the notification to be sent
+ *
+ * @returns {Boolean} true or false
+ */
+export const readNotificationService = async details => {
+  try {
+    const { notificationId } = details;
+    const readNotification = await Notification.findOne({
+      where: { id: notificationId }
+    });
+    readNotification.update({ isRead: true });
+    const response = { message: 'you have read the notification' };
+    return response;
+  } catch (error) {
+    const response = { message: 'something went wrong' };
+    return response;
+  }
+};

src/tests/controller/auth.spec.js

@@ -450,6 +450,8 @@ describe('Auth API endpoints', () => {
       expect(response.body.data.userName).to.be.equal('aboyhasnoname');
       expect(response.body.data).to.have.keys(
         'bio',
+        'id',
+        'isNotifyActive',
         'userName',
         'firstName',
         'lastName',
@@ -478,6 +480,8 @@ describe('Auth API endpoints', () => {
       expect(response.body.data).to.have.keys(
         'bio',
         'userName',
+        'id',
+        'isNotifyActive',
         'firstName',
         'lastName',
         'twitterHandle',

src/tests/controller/bookmark.spec.js

@@ -165,6 +165,7 @@ describe('Bookmarks API Endpoints', () => {
         'body',
         'image',
         'likesCount',
+        'isPublished',
         'viewsCount',
         'description',
         'bookmarks'