fix(notification-service): notification is read set to false as default

RFIT-568
sourcefuse · Nov 4, 2020 · 931de34 · 931de34
1 parent 4343a3f
commit 931de34
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/services/notification-service/src/controllers/notification-user.controller.ts b/services/notification-service/src/controllers/notification-user.controller.ts
@@ -317,7 +317,7 @@ export class NotificationUserController {
 
   private _createFilterBuilder(
     currentUser: IAuthUserWithPermissions,
-    filter?: Filter<NotificationUser>,
+    filter: Filter<NotificationUser> = {},
   ) {
     const filterBuilder = new FilterBuilder(filter);
     if (filter) {

diff --git a/services/notification-service/src/models/notification-user.model.ts b/services/notification-service/src/models/notification-user.model.ts
@@ -31,10 +31,10 @@ export class NotificationUser extends BaseEntity {
 
   @property({
     type: 'boolean',
-    required: true,
+    default: false,
     name: 'is_read',
   })
-  isRead: boolean;
+  isRead?: boolean;
 
   constructor(data?: Partial<NotificationUser>) {
     super(data);