fixes

Author: Roberto-Teigeiro

MtdrSpring/backend/frontend-service/src/main/frontend/src/components/pages/home/AssignUserDialog.tsx

@@ -18,12 +18,11 @@ import { useUserResolver } from "../../hooks/useUserResolver"; // Agregar import
 
 // Definir las interfaces basadas en los datos reales del API
 interface UserType {
-  readonly id?: string;
-  readonly userId?: string;
-  readonly name?: string;
-  readonly email?: string;
-  readonly avatar?: string;
-  readonly role?: string | null;
+  id: string;
+  name: string;
+  email: string;
+  avatar: string | null;
+  role?: string | null;
 }
 
 interface AssignUserDialogProps {
@@ -86,8 +85,12 @@ export function AssignUserDialog({
 
       const data = await response.json();
 
-      // Extraer los IDs de usuario únicos
-      const userIds = [...new Set(data.map((user: any) => user.userId).filter(Boolean))];
+      // Extraer los IDs de usuario únicos - CORREGIR AQUÍ
+      const userIds = [...new Set(
+        data
+          .filter((user: any) => user.userId && typeof user.userId === 'string')
+          .map((user: any) => user.userId)
+      )] as string[];
 
       // Resolver nombres de usuario si hay IDs
       let userNames: Record<string, string> = {};
@@ -100,11 +103,11 @@ export function AssignUserDialog({
         const resolvedName = userNames[user.userId];
 
         return {
-          id: user.userId, // Usar userId como id
-          name: resolvedName || user.name || `Usuario ${user.userId.slice(-8)}`, // Usar nombre resuelto, nombre original o fallback
-          email: user.email || "",
+          id: user.userId || '', // Asegurar que nunca sea undefined
+          name: resolvedName || user.name || `Usuario ${user.userId?.slice(-8) || ''}`, // Usar nombre resuelto, nombre original o fallback
+          email: user.email || '',
           avatar: user.avatar || null,
-          role: user.role,
+          role: user.role || null,
         };
       });
 

MtdrSpring/backend/frontend-service/src/main/frontend/src/components/pages/home/Dashboard.tsx

@@ -242,8 +242,13 @@ export default function Dashboard() {
       }, {});
       setMemberRoles(rolesMap);
 
-      // Extraer IDs de usuario únicos y resolver nombres
-      const userIds = [...new Set(members.map((member: any) => member.userId).filter(Boolean))];
+      // Extraer IDs de usuario únicos y resolver nombres - ALTERNATIVA SIMPLE
+      const userIds: string[] = [...new Set(
+        members
+          .map((member: any) => member.userId)
+          .filter(Boolean)
+      )] as string[];
+
       if (userIds.length > 0) {
         const resolvedNames = await resolveUserNames(userIds);
         setMemberNames(resolvedNames);
@@ -310,59 +315,6 @@ export default function Dashboard() {
   }, [availableTeams]);
 
   // Memoizar datos de los charts para evitar recálculos innecesarios
-  const chartData = useMemo(() => {
-    // Mover toda la lógica de los charts aquí
-    return {
-      hoursBySprintAndDeveloper: sprints.reduce((acc: Record<string, Record<string, number>>, sprint) => {
-        const currentSprintTasks = sprintTasks[sprint.sprintId] || [];
-        
-        if (!acc[sprint.name]) {
-          acc[sprint.name] = {};
-        }
-        
-        currentSprintTasks.forEach((task: Task) => {
-          if (task.assignee && task.realHours) {
-            const developerName = getUserDisplayName(task.assignee);
-            if (!acc[sprint.name][developerName]) {
-              acc[sprint.name][developerName] = 0;
-            }
-            acc[sprint.name][developerName] += task.realHours;
-          }
-        });
-        
-        return acc;
-      }, {}),
-      
-      tasksBySprintAndDeveloper: sprints.reduce((acc: Record<string, Record<string, { total: number; completed: number }>>, sprint) => {
-        const currentSprintTasks = sprintTasks[sprint.sprintId] || [];
-        
-        if (!acc[sprint.name]) {
-          acc[sprint.name] = {};
-        }
-        
-        currentSprintTasks.forEach((task: Task) => {
-          if (task.assignee) {
-            const developerName = getUserDisplayName(task.assignee);
-            if (!acc[sprint.name][developerName]) {
-              acc[sprint.name][developerName] = { total: 0, completed: 0 };
-            }
-            acc[sprint.name][developerName].total++;
-            if (task.status === "COMPLETED") {
-              acc[sprint.name][developerName].completed++;
-            }
-          }
-        });
-        
-        return acc;
-      }, {}),
-      
-      totalRealHoursBySprint: sprints.map(sprint => {
-        const tasks = sprintTasks[sprint.sprintId] || [];
-        return tasks.reduce((total, task) => total + (task.realHours || 0), 0);
-      }),
-    };
-  }, [sprints, sprintTasks, getUserDisplayName]);
-
   // Show a loading state while Clerk is initializing
   if (!isLoaded) {
     return (

MtdrSpring/backend/frontend-service/src/main/frontend/src/components/pages/home/Sprints.tsx

@@ -115,12 +115,12 @@ export default function Sprints() {
 
       const data = await response.json();
 
-      // Obtener todos los IDs de usuarios únicos que tienen asignee
-      const assigneeIds = [...new Set(data
+      // Obtener todos los IDs de usuarios únicos que tienen asignee - VERSIÓN MEJORADA
+    const assigneeIds = [...new Set(
+      data
         .map((task: ServerTask) => task.assignee)
-        .filter((assignee): assignee is string => Boolean(assignee))
-      )];
-      
+        .filter((assignee:any): assignee is string => Boolean(assignee) && typeof assignee === 'string')
+    )] as string[];
       // Resolver nombres de usuario si hay assignees
       let userNames: Record<string, string> = {};
       if (assigneeIds.length > 0) {

MtdrSpring/backend/frontend-service/src/main/frontend/src/components/pages/home/TeamManagementModal.tsx

@@ -1,6 +1,5 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { useState, useEffect } from "react";
-import React from "react";
 import { useAuth } from "@clerk/clerk-react";
 import { Users, Plus, X, UserPlus } from "lucide-react";