Merge pull request #92 from souravbhowmik1999/searchCohortApiChanges

TypeORMError: Provided take value is not a number. Please provide a n…
tekdi · Apr 3, 2024 · eabb044 · eabb044
2 parents 7b8db7d + e71a9b5
commit eabb044
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/cohort/cohort.controller.ts b/src/cohort/cohort.controller.ts
@@ -26,6 +26,8 @@ import {
   Headers,
   Delete,
   UseGuards,
+  ValidationPipe,
+  UsePipes,
 } from "@nestjs/common";
 import { CohortSearchDto } from "./dto/cohort-search.dto";
 import { Request } from "@nestjs/common";
@@ -133,6 +135,7 @@ export class CohortController {
   @ApiBody({ type: CohortSearchDto })
   @ApiForbiddenResponse({ description: "Forbidden" })
   @UseInterceptors(ClassSerializerInterceptor)
+  @UsePipes(ValidationPipe)
   @SerializeOptions({
     strategy: "excludeAll",
   })

diff --git a/src/cohort/dto/cohort-search.dto.ts b/src/cohort/dto/cohort-search.dto.ts
@@ -1,10 +1,13 @@
 import { ApiProperty, ApiPropertyOptional } from "@nestjs/swagger";
+import { IsNumberString } from "class-validator";
+
 
 export class CohortSearchDto {
   @ApiProperty({
     type: String,
     description: "Limit",
   })
+  @IsNumberString()
   limit: string;
 
   @ApiProperty({