Polish

Author: ji-mmyliu

backend/__mocks__/mockData.ts

@@ -61,6 +61,7 @@ export const testAdmins = [
     email: "peter.pan@example.com",
     status: "Active" as Status,
     role: "Administrator" as Role,
+    bookmarks: [],
   },
 ];
 
@@ -74,6 +75,7 @@ export const testFacilitators = [
     status: "Active" as Status,
     role: "Facilitator" as Role,
     learners: ["67e60671fb8fbc9c9bbb6d8a"],
+    bookmarks: [],
   },
 ];
 
@@ -93,6 +95,7 @@ export const testLearners = [
         // testCourseModules[0].pages.map(page => page.toString())
       },
     },
+    bookmarks: [],
   },
 ];
 
@@ -113,6 +116,7 @@ export const testLearnersDTO: LearnerDTO[] = [
         ]),
       ],
     ]),
+    bookmarks: [],
   },
 ];
 

backend/rest/userRoutes.ts

@@ -10,7 +10,7 @@ import {
   updateUserDtoValidator,
   uploadProfilePictureValidator,
 } from "../middlewares/validators/userValidators";
-import UserModel, { Bookmark, LearnerModel } from "../models/user.mgmodel";
+import UserModel, { Bookmark } from "../models/user.mgmodel";
 import nodemailerConfig from "../nodemailer.config";
 import AuthService from "../services/implementations/authService";
 import CoursePageService from "../services/implementations/coursePageService";

backend/services/interfaces/coursePageService.ts

@@ -17,7 +17,10 @@ interface ICoursePageService {
    * @param coursePageId the id of the page we want to fetch
    * @throwsError if course page was not successfully fetched or not found
    */
-  getCoursePage(coursePageId: string, lean?: boolean): Promise<CoursePageDTO>;
+  getCoursePage(
+    coursePageId: string,
+    lean?: boolean,
+  ): Promise<CoursePageDTOBase>;
 
   /**
    * Creates a course page, appended as the last page in the module (for now)