fix(server): rename limit to pageSize in func log api (#1202)

labring · May 31, 2023 · ac49ded · ac49ded
1 parent dbf3121
commit ac49ded
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/server/src/function/function.service.ts b/server/src/function/function.service.ts
@@ -202,12 +202,12 @@ export class FunctionService {
     appid: string,
     params: {
       page: number
-      limit: number
+      pageSize: number
       requestId?: string
       functionName?: string
     },
   ) {
-    const { page, limit, requestId, functionName } = params
+    const { page, pageSize, requestId, functionName } = params
     const { db, client } = await this.databaseService.findAndConnect(appid)
 
     try {
@@ -222,8 +222,8 @@ export class FunctionService {
 
       const data = await coll
         .find(query, {
-          limit,
-          skip: (page - 1) * limit,
+          limit: pageSize,
+          skip: (page - 1) * pageSize,
           sort: { _id: -1 },
         })
         .toArray()

diff --git a/server/src/log/log.controller.ts b/server/src/log/log.controller.ts
@@ -49,35 +49,35 @@ export class LogController {
     required: false,
   })
   @ApiQuery({
-    name: 'limit',
+    name: 'pageSize',
     type: String,
-    description: 'The limit number, default is 10',
+    description: 'The page size, default is 10',
     required: false,
   })
   @Get('functions')
   async getLogs(
     @Param('appid') appid: string,
     @Query('requestId') requestId?: string,
     @Query('functionName') functionName?: string,
-    @Query('limit') limit?: number,
+    @Query('pageSize') pageSize?: number,
     @Query('page') page?: number,
   ) {
     page = page || 1
-    limit = limit || 10
+    pageSize = pageSize || 10
 
     const res = await this.funcService.getLogs(appid, {
       requestId,
       functionName,
-      limit,
+      pageSize: pageSize,
       page,
     })
 
     return ResponseUtil.ok({
       list: res.data,
       total: res.total,
       page,
-      limit, // @deprecated use pageSize instead
-      pageSize: limit,
+      limit: pageSize, // @deprecated use pageSize instead
+      pageSize: pageSize,
     })
   }
 }