refactor: clean up logging and enhance task management features

- Removed unnecessary console logs from ProjectManagementService.
- Improved handling of optional properties in task updates to prevent errors.
- Enhanced the upload controller to include a public decorator for better access control.
- Optimized task merging logic in ProjectManagement component to ensure comments are preserved during updates.
- Added loading states in TaskModal for better user experience during file uploads and task saving.

Author: asepindrak

backend/src/project-management/project-management.service.ts

@@ -414,13 +414,6 @@ export class ProjectManagementService {
       },
     });
 
-    console.log(
-      "[createTask] created id=",
-      t.id,
-      "clientId=",
-      payload.clientId ?? null
-    );
-
     // return serialized created task (timestamps as ISO)
     return {
       ...t,
@@ -462,7 +455,7 @@ export class ProjectManagementService {
       });
       if (!project) throw new NotFoundException("Project not found");
     }
-    console.log(project);
+
     let assignee: any = null;
     // validate assignee if present and not null
     if (
@@ -569,9 +562,11 @@ export class ProjectManagementService {
     }
 
     if (assigneeId !== data.assigneeId) {
-      emailTitle = `👤 Task Assignee has Changed to ${assignee.name ?? "none"}`;
+      emailTitle = `👤 Task Assignee has Changed to ${
+        assignee?.name ?? "none"
+      }`;
       emailDescription = `👤 A task Assignee has been changed to ${
-        assignee.name ?? "none"
+        assignee?.name ?? "none"
       } on <strong>${projectName}</strong>.`;
     }
 
@@ -584,11 +579,11 @@ export class ProjectManagementService {
     Description:
     ${updated.description ?? "No description"}
 
-    Status: ${updated.status}
-    Assignee: ${assignee.name ?? "none"}
-    Priority: ${updated.priority ?? "none"}
-    Start Date: ${format(updated.startDate)}
-    Due Date: ${format(updated.dueDate)}
+    Status: ${updated?.status}
+    Assignee: ${assignee?.name ?? "none"}
+    Priority: ${updated?.priority ?? "none"}
+    Start Date: ${format(updated?.startDate)}
+    Due Date: ${format(updated?.dueDate)}
 
     Project:
     ${projectName}
@@ -623,22 +618,22 @@ export class ProjectManagementService {
 
             <p style="margin:0 0 14px; font-size:15px;">
               <strong style="color:#2b6cb0;">Status:</strong> ${
-                updated.status
+                updated?.status
               }<br>
               <strong style="color:#2b6cb0;">Assignee:</strong> ${
-                assignee.name ?? "none"
+                assignee?.name ?? "none"
               }<br>
               <strong style="color:#2b6cb0;">Priority:</strong> ${
-                updated.priority ?? "none"
+                updated?.priority ?? "none"
               }
             </p>
 
             <p style="margin:0 0 14px; font-size:15px;">
               <strong style="color:#2b6cb0;">Start Date:</strong> ${format(
-                updated.startDate
+                updated?.startDate
               )}<br>
               <strong style="color:#2b6cb0;">Due Date:</strong> ${format(
-                updated.dueDate
+                updated?.dueDate
               )}
             </p>
 
@@ -667,7 +662,6 @@ export class ProjectManagementService {
       </div>
     `;
 
-    console.log(textMsg);
     // KIRIM EMAIL
     for (const recipient of toEmails) {
       await this.email.sendMail({
@@ -1330,7 +1324,7 @@ export class ProjectManagementService {
       include: { assignee: true },
       orderBy: { createdAt: "desc" },
     });
-    console.log(tasks);
+
     const team = await prisma.teamMember.findMany({
       where: { isTrash: false },
       orderBy: { name: "asc" },

backend/src/upload/upload.controller.ts

@@ -1,53 +1,65 @@
 // src/upload/upload.controller.ts
 import {
-    Controller,
-    Post,
-    UseInterceptors,
-    UploadedFiles,
-    Body,
-    BadRequestException,
-    HttpCode,
-    HttpStatus,
-} from '@nestjs/common';
-import { FilesInterceptor } from '@nestjs/platform-express';
-import { UploadService } from './upload.service';
-import multer from 'multer';
+  Controller,
+  Post,
+  UseInterceptors,
+  UploadedFiles,
+  Body,
+  BadRequestException,
+  HttpCode,
+  HttpStatus,
+} from "@nestjs/common";
+import { FilesInterceptor } from "@nestjs/platform-express";
+import { UploadService } from "./upload.service";
+import multer from "multer";
+import { Public } from "src/auth/public.decorator";
 
-@Controller('upload')
+@Controller("upload")
 export class UploadController {
-    constructor(private readonly uploadService: UploadService) { }
+  constructor(private readonly uploadService: UploadService) {}
 
-    /**
-     * POST /upload
-     * multipart form:
-     *  - file: single or multiple files (use input name "file")
-     *  - folder: optional form field for prefix
-     *
-     * Response:
-     * { success: true, uploaded: [ { key, url }, ... ] }
-     */
-    @Post()
-    @HttpCode(HttpStatus.OK)
-    @UseInterceptors(
-        FilesInterceptor('file', 20, {
-            storage: multer.memoryStorage(),
-            limits: { fileSize: 50 * 1024 * 1024 }, // max 50MB per file (tweak as needed)
-        }),
-    )
-    async uploadFiles(@UploadedFiles() files: Express.Multer.File[], @Body('folder') folder?: string) {
-        if (!files || files.length === 0) {
-            throw new BadRequestException('No file provided. Use field name "file" in multipart form.');
-        }
+  /**
+   * POST /upload
+   * multipart form:
+   *  - file: single or multiple files (use input name "file")
+   *  - folder: optional form field for prefix
+   *
+   * Response:
+   * { success: true, uploaded: [ { key, url }, ... ] }
+   */
+  @Public()
+  @Post()
+  @HttpCode(HttpStatus.OK)
+  @UseInterceptors(
+    FilesInterceptor("file", 20, {
+      storage: multer.memoryStorage(),
+      limits: { fileSize: 50 * 1024 * 1024 }, // max 50MB per file (tweak as needed)
+    })
+  )
+  async uploadFiles(
+    @UploadedFiles() files: Express.Multer.File[],
+    @Body("folder") folder?: string
+  ) {
+    if (!files || files.length === 0) {
+      throw new BadRequestException(
+        'No file provided. Use field name "file" in multipart form.'
+      );
+    }
 
-        // sanitize folder - disallow leading ../ for safety
-        const safeFolder = folder ? folder.replace(/\.\.+/g, '').replace(/^\/+/, '') : '';
+    // sanitize folder - disallow leading ../ for safety
+    const safeFolder = folder
+      ? folder.replace(/\.\.+/g, "").replace(/^\/+/, "")
+      : "";
 
-        const uploaded = await this.uploadService.uploadMultipleFiles(files, safeFolder);
+    const uploaded = await this.uploadService.uploadMultipleFiles(
+      files,
+      safeFolder
+    );
 
-        return {
-            success: true,
-            count: uploaded.length,
-            uploaded,
-        };
-    }
+    return {
+      success: true,
+      count: uploaded.length,
+      uploaded,
+    };
+  }
 }