Fix back endpoint bug

GeneralMagicio · Aug 1, 2024 · 79324ae · 79324ae
1 parent bd4b21f
commit 79324ae
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/src/flow/dto/bodies.ts b/src/flow/dto/bodies.ts
@@ -39,9 +39,10 @@ export class DnDBody {
 
 export class RemoveLastVoteDto {
   @ApiProperty({
+    type: 'number',
     description:
-      'Collection id (skip for the top-level collection comparisons)',
+      'Collection id (null for the top-level collection comparisons)',
   })
-  @IsOptional()
-  collectionId?: number;
+  @IsDefined()
+  collectionId: number | null;
 }
diff --git a/src/flow/flow.controller.ts b/src/flow/flow.controller.ts
@@ -233,9 +233,10 @@ export class FlowController {
   @Post('/pairs/back')
   async removeLastVote(
     @Req() { userId }: AuthedReq,
-    @Body('cid') { collectionId }: RemoveLastVoteDto,
+    @Body() { collectionId }: RemoveLastVoteDto,
   ) {
-    if (!collectionId) return this.flowService.removeLastVote(userId, null);
+    if (collectionId === null)
+      return this.flowService.removeLastVote(userId, null);
 
     const progressStatus = await this.flowService.getCollectionProgressStatus(
       userId,