feat: add, update, delete card

xgeekshq · Jan 17, 2022 · 313133b · 313133b
1 parent 1f3d715
commit 313133b
Show file tree

Hide file tree

Showing 19 changed files with 6,165 additions and 4,345 deletions.
diff --git a/backend/package-lock.json b/backend/package-lock.json
diff --git a/backend/src/app.module.ts b/backend/src/app.module.ts
@@ -5,7 +5,7 @@ import DatabaseModule from './database/database.module';
 import UsersModule from './models/users/users.module';
 import AuthModule from './auth/auth.module';
 import BoardsModule from './models/boards/boards.module';
-import ActionsModule from './actions/actions.module';
+import ActionsModule from './socket/socket.module';
 
 @Module({
   imports: [

diff --git a/backend/src/auth/auth.service.ts b/backend/src/auth/auth.service.ts
@@ -28,7 +28,6 @@ export default class AuthService {
     if (!user.password)
       throw new HttpException(INVALID_CREDENTIALS, HttpStatus.BAD_REQUEST);
     await this.verifyPassword(plainTextPassword, user.password);
-    user.password = undefined;
     return user;
   }
 
@@ -49,7 +48,6 @@ export default class AuthService {
         ...registrationData,
         password: hashedPassword,
       });
-      createdUser.password = undefined;
       return createdUser;
     } catch (error) {
       if (error?.code === Errors.UniqueViolation) {

diff --git a/backend/src/models/boards/boards.controller.ts b/backend/src/models/boards/boards.controller.ts
@@ -12,14 +12,17 @@ import BoardsService from './boards.service';
 import BoardDto from './dto/board.dto';
 import JwtAuthenticationGuard from '../../guards/jwtAuth.guard';
 import RequestWithUser from '../../interfaces/requestWithUser.interface';
-import ActionsGateway from '../../actions/actions.gateway';
+import ActionsGateway from '../../socket/socket.gateway';
 import { UpdateCardPositionDto } from './dto/card/updateCardPosition.dto';
+import { AddCardDto } from './dto/card/addCard.dto';
+import DeleteCardDto from './dto/card/deleteCard.dto';
+import UpdateCardDto from './dto/card/updateCard.dto';
 
 @Controller('boards')
 export default class BoardsController {
   constructor(
     private readonly boardService: BoardsService,
-    private readonly actionsService: ActionsGateway,
+    private readonly socketService: ActionsGateway,
   ) {}
 
   // #region BOARD
@@ -41,8 +44,10 @@ export default class BoardsController {
   @UseGuards(JwtAuthenticationGuard)
   @Get(':id')
   getBoard(@Req() request: RequestWithUser) {
-    const { id: boardId } = request.params;
-    const { _id: userId } = request.user;
+    const {
+      user: { _id: userId },
+      params: { id: boardId },
+    } = request;
     return this.boardService.getBoardWithEmail(boardId, userId);
   }
 
@@ -56,27 +61,81 @@ export default class BoardsController {
   @UseGuards(JwtAuthenticationGuard)
   @Delete(':id')
   deleteBoard(@Req() request: RequestWithUser) {
-    const { id: boardId } = request.params;
-    const { _id: userId } = request.user;
+    const {
+      user: { _id: userId },
+      params: { id: boardId },
+    } = request;
     return this.boardService.deleteBoard(boardId, userId);
   }
 
   // #endregion
 
   // #region CARD
 
+  @UseGuards(JwtAuthenticationGuard)
+  @Post(':id/card')
+  async addCard(@Req() request, @Body() createCardDto: AddCardDto) {
+    const {
+      user: { _id: userId },
+      params: { id: boardId },
+    } = request;
+    const { card, colIdToAdd, socketId } = createCardDto;
+    const board = await this.boardService.addCardToBoard(
+      boardId,
+      userId,
+      card,
+      colIdToAdd,
+    );
+    this.socketService.sendUpdatedBoard(board, socketId);
+    return board;
+  }
+
+  @UseGuards(JwtAuthenticationGuard)
+  @Delete(':id/card/:cardId')
+  async deleteCard(@Req() request, @Body() deleteCardDto: DeleteCardDto) {
+    const {
+      params: { id: boardId, cardId },
+      user: { _id: userId },
+    } = request;
+    const { socketId } = deleteCardDto;
+    const board = await this.boardService.deleteCard(boardId, cardId, userId);
+    this.socketService.sendUpdatedBoard(board, socketId);
+    return board;
+  }
+
+  @UseGuards(JwtAuthenticationGuard)
+  @Put(':id/card/:cardId/items/:cardItemId')
+  async updateCardText(@Req() request, @Body() updateCardDto: UpdateCardDto) {
+    const {
+      params: { id: boardId, cardId, cardItemId },
+      user: { _id: userId },
+    } = request;
+    const { text, socketId } = updateCardDto;
+    const board = await this.boardService.updateCardText(
+      boardId,
+      cardId,
+      cardItemId,
+      userId,
+      text,
+    );
+    this.socketService.sendUpdatedBoard(board, socketId);
+    return board;
+  }
+
   @Put(':id/card/:cardId/updateCardPosition')
   async updateCardPosition(
     @Req() request,
     @Body() boardData: UpdateCardPositionDto,
   ) {
     const { id: boardId, cardId } = request.params;
+    const { targetColumnId, newPosition, socketId } = boardData;
     const board = await this.boardService.updateCardPosition(
       boardId,
       cardId,
-      boardData,
+      targetColumnId,
+      newPosition,
     );
-    this.actionsService.sendUpdatedBoard(board, boardData.socketId);
+    this.socketService.sendUpdatedBoard(board, socketId);
     return board;
   }
 

diff --git a/backend/src/models/boards/boards.module.ts b/backend/src/models/boards/boards.module.ts
@@ -1,6 +1,6 @@
 import { forwardRef, Module } from '@nestjs/common';
 import { MongooseModule } from '@nestjs/mongoose';
-import ActionsModule from '../../actions/actions.module';
+import ActionsModule from '../../socket/socket.module';
 import BoardsService from './boards.service';
 import BoardsController from './boards.controller';
 import Board, { BoardSchema } from './schemas/board.schema';

diff --git a/backend/src/models/boards/boards.service.ts b/backend/src/models/boards/boards.service.ts
@@ -12,7 +12,7 @@ import Board, { BoardDocument } from './schemas/board.schema';
 import BoardDto from './dto/board.dto';
 import { encrypt } from '../../utils/bcrypt';
 import Card from './schemas/card.schema';
-import { UpdateCardPositionDto } from './dto/card/updateCardPosition.dto';
+import CardDto from './dto/card/card.dto';
 
 @Injectable()
 export default class BoardsService {
@@ -134,10 +134,9 @@ export default class BoardsService {
   async updateCardPosition(
     boardId: string,
     cardId: string,
-    updateCardPositionDto: UpdateCardPositionDto,
+    targetColumnId: string,
+    newPosition: number,
   ) {
-    const { columnId, position } = updateCardPositionDto;
-
     const cardToMove = await this.getCardFromBoard(boardId, cardId);
 
     if (cardToMove) {
@@ -163,13 +162,13 @@ export default class BoardsService {
         .findOneAndUpdate(
           {
             _id: boardId,
-            'columns._id': columnId,
+            'columns._id': targetColumnId,
           },
           {
             $push: {
               'columns.$.cards': {
                 $each: [cardToMove],
-                $position: position,
+                $position: newPosition,
               },
             },
           },
@@ -184,5 +183,87 @@ export default class BoardsService {
     throw new HttpException(UPDATE_FAILED, HttpStatus.BAD_REQUEST);
   }
 
+  async addCardToBoard(
+    id: number,
+    userId: string,
+    card: CardDto,
+    colIdToAdd: string,
+  ) {
+    card.createdBy = userId;
+    card.items[0].createdBy = userId;
+    const result = await this.boardModel
+      .findOneAndUpdate(
+        {
+          _id: id,
+          'columns._id': colIdToAdd,
+        },
+        {
+          $push: {
+            'columns.$.cards': { ...card },
+          },
+        },
+        { new: true, rawResult: true },
+      )
+      .exec();
+    if (result.value && result.lastErrorObject?.updatedExisting)
+      return result.value;
+    throw new HttpException(INSERT_FAILED, HttpStatus.BAD_REQUEST);
+  }
+
+  async updateCardText(
+    boardId: string,
+    cardId: string,
+    cardItemId: string,
+    userId: string,
+    text: string,
+  ) {
+    const result = await this.boardModel
+      .findOneAndUpdate(
+        {
+          _id: boardId,
+          'columns.cards._id': cardId,
+          'columns.cards.createdBy': userId,
+        },
+        {
+          $set: {
+            'columns.$.cards.$[c].text': text,
+            'columns.$.cards.$[c].items.$[item].text': text,
+          },
+        },
+        {
+          arrayFilters: [{ 'c._id': cardId }, { 'item._id': cardItemId }],
+          new: true,
+          rawResult: true,
+        },
+      )
+      .exec();
+    if (result.value && result.lastErrorObject?.updatedExisting)
+      return result.value;
+    throw new HttpException(UPDATE_FAILED, HttpStatus.BAD_REQUEST);
+  }
+
+  async deleteCard(boardId: string, cardId: string, userId: string) {
+    const result = await this.boardModel
+      .findOneAndUpdate(
+        {
+          _id: boardId,
+          'columns.cards._id': cardId,
+          'columns.cards.createdBy': userId,
+        },
+        {
+          $pull: {
+            'columns.$[].cards': { _id: cardId },
+          },
+        },
+        { new: true, rawResult: true },
+      )
+      .exec();
+
+    if (result.value && result.lastErrorObject?.updatedExisting)
+      return result.value;
+
+    throw new HttpException(UPDATE_FAILED, HttpStatus.BAD_REQUEST);
+  }
+
   // #endregion
 }
diff --git a/backend/src/models/boards/dto/card/addCard.dto.ts b/backend/src/models/boards/dto/card/addCard.dto.ts
@@ -0,0 +1,17 @@
+import { Type } from 'class-transformer';
+import { IsNotEmpty, IsString } from 'class-validator';
+import CardDto from './card.dto';
+
+export class AddCardDto {
+  @IsNotEmpty()
+  @IsString()
+  colIdToAdd!: string;
+
+  @IsNotEmpty()
+  @Type(() => CardDto)
+  card!: CardDto;
+
+  @IsNotEmpty()
+  @IsString()
+  socketId!: string;
+}
diff --git a/backend/src/models/boards/dto/card/card.dto.ts b/backend/src/models/boards/dto/card/card.dto.ts
@@ -1,25 +1,8 @@
-import { IsNotEmpty, IsString, ValidateNested } from 'class-validator';
-import { Transform, TransformFnParams } from 'class-transformer';
-import CardItem from '../../schemas/card.item.schema';
-
-export default class CardDto {
-  @IsNotEmpty()
-  @IsString()
-  _id!: string;
-
-  @IsNotEmpty()
-  @IsString()
-  @Transform(({ value }: TransformFnParams) => value.trim())
-  text!: string;
+import { IsNotEmpty, ValidateNested } from 'class-validator';
+import CardItemDto from './card.item.dto';
 
+export default class CardDto extends CardItemDto {
   @IsNotEmpty()
   @ValidateNested({ each: true })
-  items!: CardItem[];
-
-  // @ValidateNested({ each: true })
-  // @Type(() => CommentDto)
-  // comments!: CommentDto[];
-
-  // @IsNotEmpty()
-  // votes!: string[];
+  items!: CardItemDto[];
 }
diff --git a/backend/src/models/boards/dto/card/card.item.dto.ts b/backend/src/models/boards/dto/card/card.item.dto.ts
@@ -1,5 +1,6 @@
-import { IsNotEmpty, IsString } from 'class-validator';
-import { Transform, TransformFnParams } from 'class-transformer';
+import { IsNotEmpty, IsString, ValidateNested } from 'class-validator';
+import { Transform, TransformFnParams, Type } from 'class-transformer';
+import CommentDto from '../comment/comment.dto';
 
 export default class CardItemDto {
   @IsNotEmpty()
@@ -11,10 +12,16 @@ export default class CardItemDto {
   @Transform(({ value }: TransformFnParams) => value.trim())
   text!: string;
 
-  // @ValidateNested({ each: true })
-  // @Type(() => CommentDto)
-  // comments!: CommentDto[];
+  @IsNotEmpty()
+  @IsString()
+  createdBy!: string;
 
-  // @IsNotEmpty()
-  // votes!: string[];
+  @IsNotEmpty()
+  @ValidateNested({ each: true })
+  @Type(() => CommentDto)
+  comments!: CommentDto[];
+
+  @IsNotEmpty()
+  @ValidateNested({ each: true })
+  votes!: string[];
 }
diff --git a/backend/src/models/boards/dto/card/deleteCard.dto.ts b/backend/src/models/boards/dto/card/deleteCard.dto.ts
@@ -0,0 +1,7 @@
+import { IsNotEmpty, IsString } from 'class-validator';
+
+export default class DeleteCardDto {
+  @IsNotEmpty()
+  @IsString()
+  socketId!: string;
+}
diff --git a/backend/src/models/boards/dto/card/updateCard.dto.ts b/backend/src/models/boards/dto/card/updateCard.dto.ts
@@ -0,0 +1,11 @@
+import { IsNotEmpty, IsString } from 'class-validator';
+
+export default class UpdateCardDto {
+  @IsNotEmpty()
+  @IsString()
+  text!: string;
+
+  @IsNotEmpty()
+  @IsString()
+  socketId!: string;
+}
diff --git a/backend/src/models/boards/dto/card/updateCardPosition.dto.ts b/backend/src/models/boards/dto/card/updateCardPosition.dto.ts
@@ -3,19 +3,11 @@ import { IsNotEmpty, IsNumber, IsString } from 'class-validator';
 export class UpdateCardPositionDto {
   @IsNotEmpty()
   @IsString()
-  columnId!: string;
-
-  @IsNotEmpty()
-  @IsString()
-  cardId!: string;
-
-  @IsNotEmpty()
-  @IsString()
-  boardId!: string;
+  targetColumnId!: string;
 
   @IsNotEmpty()
   @IsNumber()
-  position!: number;
+  newPosition!: number;
 
   @IsNotEmpty()
   @IsString()