feat(noteList): offset and limit parameters added

* added offset and limit parameters for getNoteListByCreatorId func
codex-team · Oct 22, 2023 · 90ed70a · 90ed70a
1 parent 6128b99
commit 90ed70a
Show file tree

Hide file tree

Showing 4 changed files with 39 additions and 8 deletions.
diff --git a/src/domain/service/noteList.ts b/src/domain/service/noteList.ts
@@ -23,11 +23,13 @@ export default class NoteListService {
    * Returns note list by creator id
    *
    * @param id - note creator id
+   * @param offset - number of skipped notes
+   * @param limit - number of notes to get
    * @returns { Promise<NoteList> } note
    */
-  public async getNoteListByCreatorId(id: number): Promise<NoteList> {
+  public async getNoteListByCreatorId(id: number, offset: number, limit: number): Promise<NoteList> {
     return {
-      items: await this.repository.getNoteListByCreatorId(id),
+      items: await this.repository.getNoteListByCreatorId(id, offset, limit),
     };
   }
 }
diff --git a/src/presentation/http/router/noteList.ts b/src/presentation/http/router/noteList.ts
@@ -25,15 +25,38 @@ const NoteListRouter: FastifyPluginCallback<NoteListRouterOptions> = (fastify, o
   /**
    * Get note list by userId
    */
-  fastify.get('/', {
+  fastify.get<{
+    Querystring: {
+      offset: number,
+      limit: number,
+    },
+  }>('/', {
     config: {
       policy: [
         'authRequired',
       ],
     },
+    schema: {
+      querystring: {
+        offset: {
+          type: 'number',
+        },
+        limit: {
+          type: 'number',
+          minimum: 1,
+          maximum: 30,
+        },
+      },
+    },
   }, async (request, reply) => {
-    const { userId } = request;
-    const noteList = await noteListService.getNoteListByCreatorId(userId as number);
+    const userId = request.userId as number;
+    const offset = request.query.offset;
+
+    /**
+     * if limit > 30 or limit < 1 error throws (in scedule)
+     */
+    const limit = request.query.limit;
+    const noteList = await noteListService.getNoteListByCreatorId(userId, offset, limit);
 
     return reply.send(noteList);
   });

diff --git a/src/repository/note.repository.ts b/src/repository/note.repository.ts
@@ -83,9 +83,11 @@ export default class NoteRepository {
    * Gets note list by creator id
    *
    * @param id - note creator id
+   * @param offset - number of skipped notes
+   * @param limit - number of notes to get
    * @returns { Promise<NoteList> } note
    */
-  public async getNoteListByCreatorId(id: number): Promise<Note[]> {
-    return await this.storage.getNoteListByCreatorId(id);
+  public async getNoteListByCreatorId(id: number, offset: number, limit: number): Promise<Note[]> {
+    return await this.storage.getNoteListByCreatorId(id, offset, limit);
   }
 }
diff --git a/src/repository/storage/postgres/orm/sequelize/note.ts b/src/repository/storage/postgres/orm/sequelize/note.ts
@@ -208,10 +208,14 @@ export default class NoteSequelizeStorage {
    * Gets note list by creator id
    *
    * @param creatorId - note creator id
+   * @param offset - number of skipped notes
+   * @param limit - number of notes to get
    * @returns { Promise<NoteList> } note
    */
-  public async getNoteListByCreatorId(creatorId: number): Promise<Note[]> {
+  public async getNoteListByCreatorId(creatorId: number, offset: number, limit: number): Promise<Note[]> {
     const noteList  = await this.model.findAll({
+      offset: offset,
+      limit: limit,
       where: {
         creatorId,
       },