Merge pull request #57 from CE61-KMITL/dev

Dev

.gitignore

@@ -43,4 +43,4 @@ lerna-debug.log*
 #mongo
 /mongo-data
 
-
+.scannerwork

package.json

@@ -1,7 +1,7 @@
 {
-  "name": "grader-backend",
-  "version": "0.0.1",
-  "description": "",
+  "name": "boostupxi-api",
+  "version": "1.0.1",
+  "description": "This is an API for the CE BoostUp XI.",
   "author": "",
   "private": true,
   "license": "UNLICENSED",

src/modules/auth/auth.service.ts

@@ -1,5 +1,4 @@
-import { Injectable } from '@nestjs/common';
-import { HttpException, HttpStatus } from '@nestjs/common';
+import { Injectable, HttpException, HttpStatus } from '@nestjs/common';
 import { JwtService } from '@nestjs/jwt';
 import { InjectModel } from '@nestjs/mongoose';
 import { Model } from 'mongoose';

src/modules/files/files.controller.ts

@@ -34,7 +34,7 @@ export class FilesController {
     @UploadedFiles(
       new ParseFilePipeBuilder()
         .addFileTypeValidator({
-          fileType: /(jpg|jpeg|png|zip)$/,
+          fileType: /(jpg|jpeg|png|zip)/,
         })
         .addMaxSizeValidator({
           maxSize: 1024 * 1024 * 5,

src/modules/tasks/dto/create-task.dto.ts

@@ -5,6 +5,7 @@ import {
   IsNumber,
   IsArray,
   IsNotEmpty,
+  IsOptional,
 } from 'class-validator';
 import { IFile } from 'src/shared/interfaces/file.interface';
 import { ITestCase } from 'src/shared/interfaces/testcase.interface';
@@ -50,7 +51,7 @@ export class CreateTaskDto {
     description: 'Hint of the task',
   })
   @IsString()
-  @IsNotEmpty()
+  @IsOptional()
   hint: string;
 
   @ApiProperty({
@@ -85,6 +86,6 @@ export class CreateTaskDto {
     description: 'Files of the task',
   })
   @IsArray()
-  @IsNotEmpty()
+  @IsOptional()
   files: IFile[];
 }

src/modules/tasks/tasks.controller.ts

@@ -45,7 +45,7 @@ export class TasksController {
   }
 
   @Get()
-  @Roles(Role.Auditor, Role.Staff, Role.User)
+  @Roles(Role.Auditor, Role.Staff)
   @UseGuards(JwtGuard, RolesGuard)
   async getTasks(
     @Query(
@@ -67,19 +67,12 @@ export class TasksController {
   }
 
   @Get('/:id')
-  @Roles(Role.Auditor, Role.Staff, Role.User)
+  @Roles(Role.Auditor, Role.Staff)
   @UseGuards(JwtGuard, RolesGuard)
   async getTaskById(@Param('id') id: string): Promise<TaskI> {
     return await this.tasksService.getTaskById(id);
   }
 
-  @Get('/user/:id')
-  @Roles(Role.Auditor, Role.Staff, Role.User)
-  @UseGuards(JwtGuard, RolesGuard)
-  async getTasksByUser(@Param('id') id: string): Promise<TaskI[]> {
-    return await this.tasksService.getTasksByUser(id);
-  }
-
   @Patch('/:id')
   @Roles(Role.Auditor, Role.Staff)
   @UseGuards(JwtGuard, RolesGuard)

src/modules/tasks/tasks.service.ts

@@ -66,13 +66,6 @@ export class TasksService {
     return task;
   }
 
-  async getTasksByUser(id: string): Promise<TaskI[]> {
-    const tasks = await this.taskModel.find({
-      'author.id': id,
-    });
-    return tasks;
-  }
-
   async updateTask(id: string, updateTaskDto: UpdateTaskDto) {
     const task = await this.taskModel.findById(id);