feat: Update text, comment and vote actions in the card group

backend/src/libs/dto/param/vote.group.params.ts

@@ -1,8 +1,3 @@
-import { IsMongoId, IsString } from 'class-validator';
 import { CardGroupParams } from './card.group.params';
 
-export class VoteGroupParams extends CardGroupParams {
-  @IsMongoId()
-  @IsString()
-  userId!: string;
-}
+export class VoteGroupParams extends CardGroupParams {}

backend/src/libs/utils/arrayIdToString.ts

@@ -0,0 +1,2 @@
+export const arrayIdToString = (array: string[]) =>
+  array.map((item) => item.toString());

backend/src/modules/cards/applications/update.card.application.ts

@@ -41,4 +41,18 @@ export class UpdateCardApplicationImpl implements UpdateCardApplication {
       text,
     );
   }
+
+  updateCardGroupText(
+    boardId: string,
+    cardId: string,
+    userId: string,
+    text: string,
+  ) {
+    return this.updateCardService.updateCardGroupText(
+      boardId,
+      cardId,
+      userId,
+      text,
+    );
+  }
 }

backend/src/modules/cards/controller/cards.controller.ts

@@ -19,10 +19,6 @@ import JwtAuthenticationGuard from '../../../libs/guards/jwtAuth.guard';
 import SocketGateway from '../../socket/gateway/socket.gateway';
 import { CreateCardDto } from '../dto/create.card.dto';
 import DeleteCardDto from '../dto/delete.card.dto';
-import { CreateCardParams } from '../dto/params/create.card.params';
-import { DeleteCardParams } from '../dto/params/delete.card.params';
-import { UpdateCardParams } from '../dto/params/update-position.card.params';
-import { UpdatePositionCardParams } from '../dto/params/update-text.card.params copy';
 import UpdateCardDto from '../dto/update.card.dto';
 import { UpdateCardPositionDto } from '../dto/update-position.card..dto';
 import { CreateCardApplication } from '../interfaces/applications/create.card.application.interface';
@@ -35,6 +31,9 @@ import { BaseDto } from '../../../libs/dto/base.dto';
 import { MergeCardsParams } from '../../../libs/dto/param/merge.cards.params';
 import { UnmergeCardsParams } from '../../../libs/dto/param/unmerge.cards.params';
 import UnmergeCardsDto from '../dto/unmerge.dto';
+import { BaseParam } from '../../../libs/dto/param/base.param';
+import { CardGroupParams } from '../../../libs/dto/param/card.group.params';
+import { CardItemParams } from '../../../libs/dto/param/card.item.params';
 
 @Controller('boards')
 export default class CardsController {
@@ -57,7 +56,7 @@ export default class CardsController {
   async addCard(
     @Req() request,
     @Param()
-    params: CreateCardParams,
+    params: BaseParam,
     @Body() createCardDto: CreateCardDto,
   ) {
     const {
@@ -81,7 +80,7 @@ export default class CardsController {
   async deleteCard(
     @Req() request,
     @Param()
-    params: DeleteCardParams,
+    params: CardGroupParams,
     @Body() deleteCardDto: DeleteCardDto,
   ) {
     const {
@@ -99,7 +98,7 @@ export default class CardsController {
   async updateCardText(
     @Req() request,
     @Param()
-    params: UpdateCardParams,
+    params: CardItemParams,
     @Body() updateCardDto: UpdateCardDto,
   ) {
     const {
@@ -119,11 +118,34 @@ export default class CardsController {
     return board;
   }
 
+  @UseGuards(JwtAuthenticationGuard)
+  @Put(':boardId/card/:cardId')
+  async updateCardGroupText(
+    @Req() request,
+    @Param() params: CardGroupParams,
+    @Body() updateCardDto: UpdateCardDto,
+  ) {
+    const { boardId, cardId } = params;
+    const {
+      user: { _id: userId },
+    } = request;
+    const { text } = updateCardDto;
+    const board = await this.updateCardApp.updateCardGroupText(
+      boardId,
+      cardId,
+      userId,
+      text,
+    );
+    if (!board) throw new BadRequestException(UPDATE_FAILED);
+    this.socketService.sendUpdatedBoard(board, updateCardDto.socketId);
+    return board;
+  }
+
   @Put(':boardId/card/:cardId/updateCardPosition')
   async updateCardPosition(
     @Req() request,
     @Param()
-    params: UpdatePositionCardParams,
+    params: CardGroupParams,
     @Body() boardData: UpdateCardPositionDto,
   ) {
     const { boardId, cardId } = params;

backend/src/modules/cards/interfaces/applications/update.card.application.interface.ts

@@ -15,4 +15,10 @@ export interface UpdateCardApplication {
     userId: string,
     text: string,
   ): Promise<LeanDocument<BoardDocument> | null>;
+  updateCardGroupText(
+    boardId: string,
+    cardId: string,
+    userId: string,
+    text: string,
+  ): Promise<LeanDocument<BoardDocument> | null>;
 }

backend/src/modules/cards/interfaces/services/update.card.service.interface.ts

@@ -15,4 +15,10 @@ export interface UpdateCardService {
     userId: string,
     text: string,
   ): Promise<LeanDocument<BoardDocument> | null>;
+  updateCardGroupText(
+    boardId: string,
+    cardId: string,
+    userId: string,
+    text: string,
+  ): Promise<LeanDocument<BoardDocument> | null>;
 }

backend/src/modules/cards/services/unmerge.card.service.ts

@@ -51,13 +51,15 @@ export class UnmergeCardServiceImpl implements UnmergeCardService {
       );
       if (!cardGroup) throw Error(CARD_NOT_FOUND);
 
-      const { text, comments, votes: itemVotes } = cardGroup.items[0];
-      const newComments = cardGroup.comments.concat(comments);
-      const newVotes = (cardGroup.votes as unknown as string[]).concat(
-        itemVotes as unknown as string[],
+      const items = cardGroup.items.filter(
+        (item) => item._id.toString() !== draggedCardId,
       );
-
-      if (cardGroup.items.length === 1) {
+      if (items.length === 1) {
+        const [{ text, comments, votes: itemVotes }] = items;
+        const newComments = cardGroup.comments.concat(comments);
+        const newVotes = (cardGroup.votes as unknown as string[]).concat(
+          itemVotes as unknown as string[],
+        );
         const updateResult = await this.boardModel
           .updateOne(
             {

backend/src/modules/cards/services/update.card.service.ts

@@ -92,4 +92,30 @@ export default class UpdateCardServiceImpl implements UpdateCardService {
       .lean()
       .exec();
   }
+
+  updateCardGroupText(
+    boardId: string,
+    cardId: string,
+    userId: string,
+    text: string,
+  ) {
+    return this.boardModel
+      .findOneAndUpdate(
+        {
+          _id: boardId,
+          'columns.cards._id': cardId,
+        },
+        {
+          $set: {
+            'columns.$.cards.$[c].text': text,
+          },
+        },
+        {
+          arrayFilters: [{ 'c._id': cardId, 'c.createdBy': userId }],
+          new: true,
+        },
+      )
+      .lean()
+      .exec();
+  }
 }

backend/src/modules/comments/applications/create.comment.application.ts

@@ -25,4 +25,18 @@ export class CreateCommentApplicationImpl implements CreateCommentApplication {
       text,
     );
   }
+
+  createCardGroupComment(
+    boardId: string,
+    cardId: string,
+    userId: string,
+    text: string,
+  ) {
+    return this.createCommentService.createCardGroupComment(
+      boardId,
+      cardId,
+      userId,
+      text,
+    );
+  }
 }

backend/src/modules/comments/applications/delete.comment.application.ts

@@ -17,4 +17,12 @@ export class DeleteCommentApplicationImpl implements DeleteCommentApplication {
       userId,
     );
   }
+
+  deleteCardGroupComment(boardId: string, commentId: string, userId: string) {
+    return this.deleteCommentService.deleteCardGroupComment(
+      boardId,
+      commentId,
+      userId,
+    );
+  }
 }

backend/src/modules/comments/applications/update.comment.application.ts

@@ -27,4 +27,20 @@ export class UpdateCommentApplicationImpl implements UpdateCommentApplication {
       text,
     );
   }
+
+  updateCardGroupComment(
+    boardId: string,
+    cardId: string,
+    commentId: string,
+    userId: string,
+    text: string,
+  ) {
+    return this.updateCommentService.updateCardGroupComment(
+      boardId,
+      cardId,
+      commentId,
+      userId,
+      text,
+    );
+  }
 }