Versión 1.3.0: Conexion completa del proyecto

backend/app/api/v1/endpoints/categories.py

@@ -3,53 +3,191 @@ from sqlalchemy.ext.asyncio import AsyncSession
 from sqlalchemy import select
 from pydantic import BaseModel, ConfigDict
 from typing import List, Optional
+from datetime import datetime
 import uuid
 
 from app.core.database import get_db
 from app.models.category import Category
+from app.models.user import User
 from app.api import deps 
 
 router = APIRouter()
 
-class CategoryBase(BaseModel):
+# ===================================
+# PYDANTIC SCHEMAS
+# ===================================
+
+class CategoryCreate(BaseModel):
+    """Schema para crear categoría - NO incluye tenant_id (se asigna automáticamente)"""
     name: str
     description: Optional[str] = None
-    is_active: bool = True
+    color: Optional[str] = None
-    tenant_id: Optional[uuid.UUID] = None
+    sla_response_hours: int = 24
+    sla_resolution_hours: int = 72
+    auto_assign_to: Optional[uuid.UUID] = None
     
-class CategoryCreate(CategoryBase):
+class CategoryUpdate(BaseModel):
-    pass
+    """Schema para actualizar categoría"""
-
-class CategoryUpdate(CategoryBase):
     name: Optional[str] = None
     description: Optional[str] = None
+    color: Optional[str] = None
+    sla_response_hours: Optional[int] = None
+    sla_resolution_hours: Optional[int] = None
+    auto_assign_to: Optional[uuid.UUID] = None
     is_active: Optional[bool] = None
-    tenant_id: Optional[uuid.UUID] = None
 
-class CategoryResponse(CategoryBase):
+class CategoryResponse(BaseModel):
+    """Schema de respuesta - incluye todos los campos"""
     id: uuid.UUID
+    tenant_id: uuid.UUID  # ✅ AÑADIDO
+    name: str
+    description: Optional[str] = None
+    color: Optional[str] = None
+    sla_response_hours: int
+    sla_resolution_hours: int
+    auto_assign_to: Optional[uuid.UUID] = None
+    is_active: bool
+    created_at: datetime  # ✅ AÑADIDO
+    updated_at: datetime  # ✅ AÑADIDO
     
     model_config = ConfigDict(from_attributes=True)
 
+
+# ===================================
+# ENDPOINTS
+# ===================================
+
 @router.get("/", response_model=List[CategoryResponse])
 async def read_categories(
     skip: int = 0, 
     limit: int = 100, 
     db: AsyncSession = Depends(get_db),
-    current_user = Depends(deps.get_current_active_superuser)
+    current_user: User = Depends(deps.get_current_user)  # ✅ CORREGIDO: Type hint + no solo superuser
 ):
-    query = select(Category).offset(skip).limit(limit)
+    """
+    Listar categorías del tenant del usuario actual.
+    
+    ✅ Implementa multi-tenancy: solo muestra categorías del tenant del usuario.
+    """
+    # ✅ CORREGIDO: Filtrar por tenant_id
+    query = select(Category).where(
+        Category.tenant_id == current_user.tenant_id
+    ).offset(skip).limit(limit)
+    
     result = await db.execute(query)
     return result.scalars().all()
 
-@router.post("/", response_model=CategoryResponse)
+
+@router.post("/", response_model=CategoryResponse, status_code=status.HTTP_201_CREATED)
 async def create_category(
     category: CategoryCreate, 
     db: AsyncSession = Depends(get_db),
-    current_user = Depends(deps.get_current_active_superuser)
+    current_user: User = Depends(deps.get_current_user)  # ✅ CORREGIDO: Type hint
 ):
-    db_category = Category(**category.model_dump())
+    """
+    Crear nueva categoría en el tenant del usuario actual.
+    
+    ✅ Implementa multi-tenancy: asigna automáticamente tenant_id del usuario.
+    """
+    # ✅ CORREGIDO: Asignar tenant_id del usuario actual
+    db_category = Category(
+        **category.model_dump(),
+        tenant_id=current_user.tenant_id  # ✅ Multi-tenancy automático
+    )
+    
     db.add(db_category)
     await db.commit()
     await db.refresh(db_category)
     return db_category
+
+
+@router.get("/{category_id}", response_model=CategoryResponse)
+async def read_category(
+    category_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Obtener una categoría específica del tenant.
+    
+    ✅ Implementa multi-tenancy: solo permite acceso a categorías del propio tenant.
+    """
+    query = select(Category).where(
+        Category.id == category_id,
+        Category.tenant_id == current_user.tenant_id  # ✅ Seguridad multi-tenant
+    )
+    result = await db.execute(query)
+    category = result.scalar_one_or_none()
+    
+    if not category:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="Category not found"
+        )
+    
+    return category
+
+
+@router.put("/{category_id}", response_model=CategoryResponse)
+async def update_category(
+    category_id: uuid.UUID,
+    category_update: CategoryUpdate,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Actualizar categoría del tenant.
+    
+    ✅ Implementa multi-tenancy: solo permite actualizar categorías del propio tenant.
+    """
+    query = select(Category).where(
+        Category.id == category_id,
+        Category.tenant_id == current_user.tenant_id
+    )
+    result = await db.execute(query)
+    db_category = result.scalar_one_or_none()
+    
+    if not db_category:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="Category not found"
+        )
+    
+    # Actualizar campos
+    update_data = category_update.model_dump(exclude_unset=True)
+    for field, value in update_data.items():
+        setattr(db_category, field, value)
+    
+    await db.commit()
+    await db.refresh(db_category)
+    return db_category
+
+
+@router.delete("/{category_id}", status_code=status.HTTP_204_NO_CONTENT)
+async def delete_category(
+    category_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Desactivar categoría del tenant (soft delete).
+    
+    ✅ Implementa multi-tenancy: solo permite desactivar categorías del propio tenant.
+    """
+    query = select(Category).where(
+        Category.id == category_id,
+        Category.tenant_id == current_user.tenant_id
+    )
+    result = await db.execute(query)
+    db_category = result.scalar_one_or_none()
+    
+    if not db_category:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="Category not found"
+        )
+    
+    # Soft delete
+    db_category.is_active = False
+    await db.commit()
+    return None

backend/app/api/v1/endpoints/systems.py

@@ -3,51 +3,179 @@ from sqlalchemy.ext.asyncio import AsyncSession
 from sqlalchemy import select
 from pydantic import BaseModel, ConfigDict
 from typing import List, Optional
+from datetime import datetime
 import uuid
 
 from app.core.database import get_db
 from app.models.system import System
+from app.models.user import User
 from app.api import deps 
 
 router = APIRouter()
 
-class SystemBase(BaseModel):
+# ===================================
+# PYDANTIC SCHEMAS
+# ===================================
+
+class SystemCreate(BaseModel):
+    """Schema para crear sistema - NO incluye tenant_id (se asigna automáticamente)"""
     name: str
     description: Optional[str] = None
-    is_active: bool = True
     
-class SystemCreate(SystemBase):
+class SystemUpdate(BaseModel):
-    pass
+    """Schema para actualizar sistema"""
-
-class SystemUpdate(SystemBase):
     name: Optional[str] = None
     description: Optional[str] = None
     is_active: Optional[bool] = None
 
-class SystemResponse(SystemBase):
+class SystemResponse(BaseModel):
+    """Schema de respuesta - incluye todos los campos"""
     id: uuid.UUID
+    tenant_id: uuid.UUID  # ✅ AÑADIDO
+    name: str
+    description: Optional[str] = None
+    is_active: bool
+    created_at: datetime  # ✅ AÑADIDO
+    updated_at: datetime  # ✅ AÑADIDO
     
     model_config = ConfigDict(from_attributes=True)
 
+
+# ===================================
+# ENDPOINTS
+# ===================================
+
 @router.get("/", response_model=List[SystemResponse])
 async def read_systems(
     skip: int = 0, 
     limit: int = 100, 
     db: AsyncSession = Depends(get_db),
-    current_user = Depends(deps.get_current_active_superuser)
+    current_user: User = Depends(deps.get_current_user)  # ✅ CORREGIDO: Type hint + no solo superuser
 ):
-    query = select(System).offset(skip).limit(limit)
+    """
+    Listar sistemas del tenant del usuario actual.
+    
+    ✅ Implementa multi-tenancy: solo muestra sistemas del tenant del usuario.
+    """
+    # ✅ CORREGIDO: Filtrar por tenant_id
+    query = select(System).where(
+        System.tenant_id == current_user.tenant_id
+    ).offset(skip).limit(limit)
+    
     result = await db.execute(query)
     return result.scalars().all()
 
-@router.post("/", response_model=SystemResponse)
+
+@router.post("/", response_model=SystemResponse, status_code=status.HTTP_201_CREATED)
 async def create_system(
     system: SystemCreate, 
     db: AsyncSession = Depends(get_db),
-    current_user = Depends(deps.get_current_active_superuser)
+    current_user: User = Depends(deps.get_current_user)  # ✅ CORREGIDO: Type hint
 ):
-    db_system = System(**system.model_dump())
+    """
+    Crear nuevo sistema en el tenant del usuario actual.
+    
+    ✅ Implementa multi-tenancy: asigna automáticamente tenant_id del usuario.
+    """
+    # ✅ CORREGIDO: Asignar tenant_id del usuario actual
+    db_system = System(
+        **system.model_dump(),
+        tenant_id=current_user.tenant_id  # ✅ Multi-tenancy automático
+    )
+    
     db.add(db_system)
     await db.commit()
     await db.refresh(db_system)
     return db_system
+
+
+@router.get("/{system_id}", response_model=SystemResponse)
+async def read_system(
+    system_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Obtener un sistema específico del tenant.
+    
+    ✅ Implementa multi-tenancy: solo permite acceso a sistemas del propio tenant.
+    """
+    query = select(System).where(
+        System.id == system_id,
+        System.tenant_id == current_user.tenant_id  # ✅ Seguridad multi-tenant
+    )
+    result = await db.execute(query)
+    system = result.scalar_one_or_none()
+    
+    if not system:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="System not found"
+        )
+    
+    return system
+
+
+@router.put("/{system_id}", response_model=SystemResponse)
+async def update_system(
+    system_id: uuid.UUID,
+    system_update: SystemUpdate,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Actualizar sistema del tenant.
+    
+    ✅ Implementa multi-tenancy: solo permite actualizar sistemas del propio tenant.
+    """
+    query = select(System).where(
+        System.id == system_id,
+        System.tenant_id == current_user.tenant_id
+    )
+    result = await db.execute(query)
+    db_system = result.scalar_one_or_none()
+    
+    if not db_system:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="System not found"
+        )
+    
+    # Actualizar campos
+    update_data = system_update.model_dump(exclude_unset=True)
+    for field, value in update_data.items():
+        setattr(db_system, field, value)
+    
+    await db.commit()
+    await db.refresh(db_system)
+    return db_system
+
+
+@router.delete("/{system_id}", status_code=status.HTTP_204_NO_CONTENT)
+async def delete_system(
+    system_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Desactivar sistema del tenant (soft delete).
+    
+    ✅ Implementa multi-tenancy: solo permite desactivar sistemas del propio tenant.
+    """
+    query = select(System).where(
+        System.id == system_id,
+        System.tenant_id == current_user.tenant_id
+    )
+    result = await db.execute(query)
+    db_system = result.scalar_one_or_none()
+    
+    if not db_system:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="System not found"
+        )
+    
+    # Soft delete
+    db_system.is_active = False
+    await db.commit()
+    return None

backend/app/api/v1/endpoints/tenants.py

@@ -85,6 +85,9 @@ async def update_tenant(
         raise HTTPException(status_code=404, detail="Tenant not found")
         
     update_data = tenant_in.model_dump(exclude_unset=True)
+    if "status" in update_data:
+        tenant.is_active = update_data.pop("status") == TenantStatus.active
+
     for field, value in update_data.items():
         setattr(tenant, field, value)
         
@@ -92,3 +95,18 @@ async def update_tenant(
     await db.commit()
     await db.refresh(tenant)
     return tenant
+
+@router.delete("/{tenant_id}", status_code=status.HTTP_204_NO_CONTENT)
+async def delete_tenant(
+    tenant_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+    current_user = Depends(deps.get_current_active_superuser)
+):
+    """Eliminar un cliente (tenant) por ID."""
+    tenant = await db.get(Tenant, tenant_id)
+    if not tenant:
+        raise HTTPException(status_code=404, detail="Tenant not found")
+
+    await db.delete(tenant)
+    await db.commit()
+    return {"message": "Tenant deleted successfully"}

backend/app/api/v1/endpoints/tickets.py

@@ -0,0 +1,451 @@
+"""
+Tickets endpoints - ServiceManagerWeb
+"""
+
+from fastapi import APIRouter, Depends, HTTPException, status, UploadFile, File
+from pydantic import BaseModel
+from sqlalchemy.ext.asyncio import AsyncSession
+from sqlalchemy import select, func
+from typing import List, Optional
+from datetime import datetime
+from app.core.database import get_db
+from app.api.deps import get_current_user
+from app.models.ticket import Ticket, TicketStatus, TicketPriority
+from app.models.user import User
+from app.models.category import Category  # ✅ CORREGIDO: Era TicketCategory
+from app.models.system import System
+import uuid
+
+router = APIRouter()
+
+# ===================================
+# SCHEMAS
+# ===================================
+
+class TicketCreate(BaseModel):
+    subject: str
+    description: str
+    category_id: Optional[str] = None
+    affected_system_id: Optional[str] = None  # ✅ CORREGIDO: Era system_id
+    priority: str = "MEDIUM"
+
+class TicketUpdate(BaseModel):
+    subject: Optional[str] = None
+    description: Optional[str] = None
+    status: Optional[str] = None
+    priority: Optional[str] = None
+    assigned_to: Optional[str] = None
+
+class TicketResponse(BaseModel):
+    id: str
+    ticket_number: str
+    subject: str
+    description: str
+    status: str
+    priority: str
+    category_id: Optional[str] = None
+    affected_system_id: Optional[str] = None  # ✅ CORREGIDO: Era system_id
+    created_by: str
+    assigned_to: Optional[str] = None
+    created_at: datetime
+    updated_at: datetime
+    
+    class Config:
+        from_attributes = True
+
+class TicketCloseRequest(BaseModel):
+    resolution: Optional[str] = None
+
+
+# ===================================
+# TICKET ENDPOINTS
+# ===================================
+
+@router.post("/", response_model=TicketResponse, status_code=status.HTTP_201_CREATED)
+async def create_ticket(
+    ticket: TicketCreate,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Crear un nuevo ticket
+    """
+    try:
+        # Generar número de ticket único
+        result = await db.execute(
+            select(func.count(Ticket.id)).where(Ticket.tenant_id == current_user.tenant_id)
+        )
+        count = result.scalar() or 0
+        ticket_number = f"TK-{count + 1:06d}"
+        
+        # Convertir IDs de string a UUID si son proporcionados
+        category_uuid = uuid.UUID(ticket.category_id) if ticket.category_id else None
+        system_uuid = uuid.UUID(ticket.affected_system_id) if ticket.affected_system_id else None  # ✅ CORREGIDO
+        
+        # ✅ CORREGIDO: Validar en la tabla correcta con el nombre correcto del modelo
+        if category_uuid:
+            category = await db.get(Category, category_uuid)  # ✅ Category, no TicketCategory
+            if not category:
+                raise HTTPException(
+                    status_code=status.HTTP_400_BAD_REQUEST,
+                    detail=f"La categoría con ID {ticket.category_id} no existe."
+                )
+
+        # Validar si el system_id existe en la tabla affected_systems
+        if system_uuid:
+            system = await db.get(System, system_uuid)
+            if not system:
+                raise HTTPException(
+                    status_code=status.HTTP_400_BAD_REQUEST,
+                    detail=f"El sistema con ID {ticket.affected_system_id} no existe."
+                )
+        
+        db_ticket = Ticket(
+            id=uuid.uuid4(),
+            tenant_id=current_user.tenant_id,
+            ticket_number=ticket_number,
+            subject=ticket.subject,
+            description=ticket.description,
+            category_id=category_uuid,
+            affected_system_id=system_uuid,  # ✅ CORREGIDO: Nombre correcto del campo
+            priority=TicketPriority[ticket.priority.upper()],
+            created_by=current_user.id,
+            status=TicketStatus.NEW,
+            created_at=datetime.utcnow(),
+            updated_at=datetime.utcnow()
+        )
+        
+        db.add(db_ticket)
+        await db.commit()
+        await db.refresh(db_ticket)
+        
+        # ✅ CORREGIDO: Usar affected_system_id en respuesta
+        return {
+            "id": str(db_ticket.id),
+            "ticket_number": db_ticket.ticket_number,
+            "subject": db_ticket.subject,
+            "description": db_ticket.description,
+            "status": db_ticket.status.value,
+            "priority": db_ticket.priority.value,
+            "category_id": str(db_ticket.category_id) if db_ticket.category_id else None,
+            "affected_system_id": str(db_ticket.affected_system_id) if db_ticket.affected_system_id else None,  # ✅ CORREGIDO
+            "created_by": str(db_ticket.created_by),
+            "assigned_to": str(db_ticket.assigned_to) if db_ticket.assigned_to else None,
+            "created_at": db_ticket.created_at,
+            "updated_at": db_ticket.updated_at
+        }
+        
+    except ValueError as e:
+        await db.rollback()
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail=f"Invalid UUID format: {str(e)}"
+        )
+    except Exception as e:
+        await db.rollback()
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail=f"Error creating ticket: {str(e)}"
+        )
+
+
+@router.get("/", response_model=List[TicketResponse])
+async def get_tickets(
+    skip: int = 0,
+    limit: int = 100,
+    status_filter: Optional[str] = None,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Obtener tickets del usuario actual
+    """
+    query = select(Ticket).where(
+        Ticket.tenant_id == current_user.tenant_id,
+        Ticket.created_by == current_user.id
+    )
+    
+    if status_filter:
+        try:
+            status_enum = TicketStatus[status_filter.upper()]
+            query = query.where(Ticket.status == status_enum)
+        except KeyError:
+            raise HTTPException(
+                status_code=status.HTTP_400_BAD_REQUEST,
+                detail=f"Invalid status: {status_filter}"
+            )
+    
+    query = query.order_by(Ticket.created_at.desc()).offset(skip).limit(limit)
+    
+    result = await db.execute(query)
+    tickets = result.scalars().all()
+    
+    # ✅ CORREGIDO: Usar affected_system_id
+    return [
+        {
+            "id": str(t.id),
+            "ticket_number": t.ticket_number,
+            "subject": t.subject,
+            "description": t.description,
+            "status": t.status.value,
+            "priority": t.priority.value,
+            "category_id": str(t.category_id) if t.category_id else None,
+            "affected_system_id": str(t.affected_system_id) if t.affected_system_id else None,  # ✅ CORREGIDO
+            "created_by": str(t.created_by),
+            "assigned_to": str(t.assigned_to) if t.assigned_to else None,
+            "created_at": t.created_at,
+            "updated_at": t.updated_at
+        }
+        for t in tickets
+    ]
+
+
+@router.get("/{ticket_id}", response_model=TicketResponse)
+async def get_ticket(
+    ticket_id: str,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Obtener un ticket específico
+    """
+    try:
+        ticket_uuid = uuid.UUID(ticket_id)
+    except ValueError:
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail="Invalid ticket ID format"
+        )
+    
+    query = select(Ticket).where(
+        Ticket.id == ticket_uuid,
+        Ticket.tenant_id == current_user.tenant_id,
+        Ticket.created_by == current_user.id
+    )
+    
+    result = await db.execute(query)
+    ticket = result.scalars().first()
+    
+    if not ticket:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail=f"Ticket {ticket_id} not found"
+        )
+    
+    # ✅ CORREGIDO: Usar affected_system_id
+    return {
+        "id": str(ticket.id),
+        "ticket_number": ticket.ticket_number,
+        "subject": ticket.subject,
+        "description": ticket.description,
+        "status": ticket.status.value,
+        "priority": ticket.priority.value,
+        "category_id": str(ticket.category_id) if ticket.category_id else None,
+        "affected_system_id": str(ticket.affected_system_id) if ticket.affected_system_id else None,  # ✅ CORREGIDO
+        "created_by": str(ticket.created_by),
+        "assigned_to": str(ticket.assigned_to) if ticket.assigned_to else None,
+        "created_at": ticket.created_at,
+        "updated_at": ticket.updated_at
+    }
+
+
+@router.patch("/{ticket_id}", response_model=TicketResponse)
+async def update_ticket(
+    ticket_id: str,
+    ticket_update: TicketUpdate,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Actualizar un ticket
+    """
+    try:
+        ticket_uuid = uuid.UUID(ticket_id)
+    except ValueError:
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail="Invalid ticket ID format"
+        )
+    
+    query = select(Ticket).where(
+        Ticket.id == ticket_uuid,
+        Ticket.tenant_id == current_user.tenant_id,
+        Ticket.created_by == current_user.id
+    )
+    
+    result = await db.execute(query)
+    db_ticket = result.scalars().first()
+    
+    if not db_ticket:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail=f"Ticket {ticket_id} not found"
+        )
+    
+    try:
+        update_data = ticket_update.dict(exclude_unset=True)
+        
+        for field, value in update_data.items():
+            if field == "status" and value:
+                setattr(db_ticket, field, TicketStatus[value.upper()])
+            elif field == "priority" and value:
+                setattr(db_ticket, field, TicketPriority[value.upper()])
+            elif field == "assigned_to" and value:
+                setattr(db_ticket, field, uuid.UUID(value))
+            else:
+                setattr(db_ticket, field, value)
+        
+        db_ticket.updated_at = datetime.utcnow()
+        
+        await db.commit()
+        await db.refresh(db_ticket)
+        
+        # ✅ CORREGIDO: Usar affected_system_id
+        return {
+            "id": str(db_ticket.id),
+            "ticket_number": db_ticket.ticket_number,
+            "subject": db_ticket.subject,
+            "description": db_ticket.description,
+            "status": db_ticket.status.value,
+            "priority": db_ticket.priority.value,
+            "category_id": str(db_ticket.category_id) if db_ticket.category_id else None,
+            "affected_system_id": str(db_ticket.affected_system_id) if db_ticket.affected_system_id else None,  # ✅ CORREGIDO
+            "created_by": str(db_ticket.created_by),
+            "assigned_to": str(db_ticket.assigned_to) if db_ticket.assigned_to else None,
+            "created_at": db_ticket.created_at,
+            "updated_at": db_ticket.updated_at
+        }
+        
+    except Exception as e:
+        await db.rollback()
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail=f"Error updating ticket: {str(e)}"
+        )
+
+
+@router.patch("/{ticket_id}/close", response_model=TicketResponse)
+async def close_ticket(
+    ticket_id: str,
+    close_request: TicketCloseRequest,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Cerrar un ticket
+    """
+    try:
+        ticket_uuid = uuid.UUID(ticket_id)
+    except ValueError:
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail="Invalid ticket ID format"
+        )
+    
+    query = select(Ticket).where(
+        Ticket.id == ticket_uuid,
+        Ticket.tenant_id == current_user.tenant_id,
+        Ticket.created_by == current_user.id
+    )
+    
+    result = await db.execute(query)
+    db_ticket = result.scalars().first()
+    
+    if not db_ticket:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail=f"Ticket {ticket_id} not found"
+        )
+    
+    try:
+        db_ticket.status = TicketStatus.CLOSED
+        db_ticket.updated_at = datetime.utcnow()
+        
+        await db.commit()
+        await db.refresh(db_ticket)
+        
+        # ✅ CORREGIDO: Usar affected_system_id
+        return {
+            "id": str(db_ticket.id),
+            "ticket_number": db_ticket.ticket_number,
+            "subject": db_ticket.subject,
+            "description": db_ticket.description,
+            "status": db_ticket.status.value,
+            "priority": db_ticket.priority.value,
+            "category_id": str(db_ticket.category_id) if db_ticket.category_id else None,
+            "affected_system_id": str(db_ticket.affected_system_id) if db_ticket.affected_system_id else None,  # ✅ CORREGIDO
+            "created_by": str(db_ticket.created_by),
+            "assigned_to": str(db_ticket.assigned_to) if db_ticket.assigned_to else None,
+            "created_at": db_ticket.created_at,
+            "updated_at": db_ticket.updated_at
+        }
+        
+    except Exception as e:
+        await db.rollback()
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail=f"Error closing ticket: {str(e)}"
+        )
+
+
+# ===================================
+# COMMENT ENDPOINTS (placeholder)
+# ===================================
+
+@router.get("/{ticket_id}/comments")
+async def get_ticket_comments(
+    ticket_id: str,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Obtener comentarios de un ticket
+    """
+    return []
+
+
+@router.post("/{ticket_id}/comments", status_code=status.HTTP_201_CREATED)
+async def create_comment(
+    ticket_id: str,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Agregar un comentario a un ticket
+    """
+    raise HTTPException(
+        status_code=status.HTTP_501_NOT_IMPLEMENTED,
+        detail="Comments not yet implemented"
+    )
+
+
+# ===================================
+# ATTACHMENT ENDPOINTS (placeholder)
+# ===================================
+
+@router.get("/{ticket_id}/attachments")
+async def get_ticket_attachments(
+    ticket_id: str,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Obtener adjuntos de un ticket
+    """
+    return []
+
+
+@router.post("/{ticket_id}/attachments", status_code=status.HTTP_201_CREATED)
+async def upload_attachment(
+    ticket_id: str,
+    file: UploadFile = File(...),
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(get_current_user)
+):
+    """
+    Subir un archivo adjunto a un ticket
+    """
+    raise HTTPException(
+        status_code=status.HTTP_501_NOT_IMPLEMENTED,
+        detail="File uploads not yet implemented"
+    )

backend/app/api/v1/endpoints/users.py

@@ -3,6 +3,7 @@ from sqlalchemy.ext.asyncio import AsyncSession
 from sqlalchemy import select
 from pydantic import BaseModel, ConfigDict, EmailStr
 from typing import List, Optional
+from datetime import datetime
 import uuid
 
 from app.core.database import get_db
@@ -12,57 +13,335 @@ from app.api import deps
 
 router = APIRouter()
 
-class UserBase(BaseModel):
+# ===================================
+# PYDANTIC SCHEMAS
+# ===================================
+
+class UserCreate(BaseModel):
+    """Schema para crear usuario - NO incluye tenant_id (se asigna automáticamente)"""
     email: EmailStr
     first_name: str
     last_name: str
     role: UserRole
-    is_active: bool = True
-    tenant_id: Optional[uuid.UUID] = None
-
-class UserCreate(UserBase):
     password: str
-
+    language: str = "es"
+    timezone: str = "UTC"
+    notifications_email: bool = True
+    
 class UserUpdate(BaseModel):
+    """Schema para actualizar usuario"""
     email: Optional[EmailStr] = None
     first_name: Optional[str] = None
     last_name: Optional[str] = None
     role: Optional[UserRole] = None
     is_active: Optional[bool] = None
-    password: Optional[str] = None # Optional password update
+    password: Optional[str] = None
+    language: Optional[str] = None
+    timezone: Optional[str] = None
+    notifications_email: Optional[bool] = None
 
-class UserResponse(UserBase):
+class UserResponse(BaseModel):
+    """Schema de respuesta - incluye todos los campos públicos"""
     id: uuid.UUID
+    tenant_id: uuid.UUID
+    email: EmailStr
+    first_name: str
+    last_name: str
+    avatar_url: Optional[str] = None
+    role: UserRole
+    is_active: bool
+    email_verified: bool
+    last_login: Optional[datetime] = None
+    language: str
+    timezone: str
+    notifications_email: bool
+    totp_enabled: bool
+    created_at: datetime
+    updated_at: datetime
     
     model_config = ConfigDict(from_attributes=True)
 
+
+# ===================================
+# ENDPOINTS
+# ===================================
+
 @router.get("/", response_model=List[UserResponse])
 async def read_users(
     skip: int = 0, 
-    limit: int = 100, 
+    limit: int = 100,
+    role: Optional[UserRole] = None,
+    is_active: Optional[bool] = None,
     db: AsyncSession = Depends(get_db),
-    current_user = Depends(deps.get_current_active_superuser)
+    current_user: User = Depends(deps.get_current_user)
 ):
-    query = select(User).offset(skip).limit(limit)
+    """
+    Listar usuarios del tenant del usuario actual.
+    
+    ✅ Implementa multi-tenancy: solo muestra usuarios del tenant del usuario.
+    
+    Filtros opcionales:
+    - role: filtrar por rol
+    - is_active: filtrar por estado activo
+    """
+    # ✅ CORREGIDO: Filtrar por tenant_id
+    query = select(User).where(User.tenant_id == current_user.tenant_id)
+    
+    # Aplicar filtros opcionales
+    if role:
+        query = query.where(User.role == role)
+    if is_active is not None:
+        query = query.where(User.is_active == is_active)
+    
+    query = query.offset(skip).limit(limit).order_by(User.created_at.desc())
+    
     result = await db.execute(query)
     return result.scalars().all()
 
-@router.post("/", response_model=UserResponse)
+
+@router.post("/", response_model=UserResponse, status_code=status.HTTP_201_CREATED)
 async def create_user(
     user: UserCreate, 
     db: AsyncSession = Depends(get_db),
-    current_user = Depends(deps.get_current_active_superuser)
+    current_user: User = Depends(deps.get_current_user)
 ):
-    query = select(User).where(User.email == user.email)
+    """
+    Crear nuevo usuario en el tenant del usuario actual.
+    
+    ✅ Implementa multi-tenancy: asigna automáticamente tenant_id del usuario.
+    
+    Restricciones:
+    - Solo ADMIN, SUPPORT_MANAGER y CLIENT_ADMIN pueden crear usuarios
+    - El email debe ser único dentro del tenant
+    """
+    # Verificar permisos
+    if not current_user.can_manage_users:
+        raise HTTPException(
+            status_code=status.HTTP_403_FORBIDDEN,
+            detail="You don't have permission to create users"
+        )
+    
+    # Verificar si el email ya existe en el tenant
+    query = select(User).where(
+        User.email == user.email,
+        User.tenant_id == current_user.tenant_id
+    )
     result = await db.execute(query)
     if result.scalar_one_or_none():
-        raise HTTPException(status_code=400, detail="Email already registered")
+        raise HTTPException(
-        
+            status_code=status.HTTP_400_BAD_REQUEST,
-    user_data = user.model_dump(exclude={"password"})
+            detail="Email already registered in this tenant"
-    password_hash = security.get_password_hash(user.password)
+        )
+    
+    # Preparar datos del usuario
+    user_data = user.model_dump(exclude={"password"})
+    password_hash = security.hash_password(user.password)
+    
+    # ✅ CORREGIDO: Asignar tenant_id del usuario actual
+    db_user = User(
+        **user_data,
+        password_hash=password_hash,
+        tenant_id=current_user.tenant_id  # ✅ Multi-tenancy automático
+    )
     
-    db_user = User(**user_data, password_hash=password_hash)
     db.add(db_user)
     await db.commit()
     await db.refresh(db_user)
     return db_user
+
+
+@router.get("/{user_id}", response_model=UserResponse)
+async def read_user(
+    user_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Obtener un usuario específico del tenant.
+    
+    ✅ Implementa multi-tenancy: solo permite acceso a usuarios del propio tenant.
+    """
+    query = select(User).where(
+        User.id == user_id,
+        User.tenant_id == current_user.tenant_id  # ✅ Seguridad multi-tenant
+    )
+    result = await db.execute(query)
+    user = result.scalar_one_or_none()
+    
+    if not user:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="User not found"
+        )
+    
+    return user
+
+
+@router.put("/{user_id}", response_model=UserResponse)
+async def update_user(
+    user_id: uuid.UUID,
+    user_update: UserUpdate,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Actualizar usuario del tenant.
+    
+    ✅ Implementa multi-tenancy: solo permite actualizar usuarios del propio tenant.
+    
+    Restricciones:
+    - Solo ADMIN, SUPPORT_MANAGER y CLIENT_ADMIN pueden actualizar usuarios
+    - No se puede cambiar el tenant_id
+    """
+    # Verificar permisos
+    if not current_user.can_manage_users:
+        raise HTTPException(
+            status_code=status.HTTP_403_FORBIDDEN,
+            detail="You don't have permission to update users"
+        )
+    
+    # Buscar usuario
+    query = select(User).where(
+        User.id == user_id,
+        User.tenant_id == current_user.tenant_id
+    )
+    result = await db.execute(query)
+    db_user = result.scalar_one_or_none()
+    
+    if not db_user:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="User not found"
+        )
+    
+    # Verificar email único si se está cambiando
+    update_data = user_update.model_dump(exclude_unset=True)
+    if "email" in update_data and update_data["email"] != db_user.email:
+        email_query = select(User).where(
+            User.email == update_data["email"],
+            User.tenant_id == current_user.tenant_id,
+            User.id != user_id
+        )
+        email_result = await db.execute(email_query)
+        if email_result.scalar_one_or_none():
+            raise HTTPException(
+                status_code=status.HTTP_400_BAD_REQUEST,
+                detail="Email already in use by another user"
+            )
+    
+    # Actualizar campos
+    for field, value in update_data.items():
+        if field == "password":
+            # Hash the new password
+            db_user.password_hash = security.hash_password(value)
+        else:
+            setattr(db_user, field, value)
+    
+    await db.commit()
+    await db.refresh(db_user)
+    return db_user
+
+
+@router.delete("/{user_id}", status_code=status.HTTP_204_NO_CONTENT)
+async def delete_user(
+    user_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Desactivar usuario del tenant (soft delete).
+    
+    ✅ Implementa multi-tenancy: solo permite desactivar usuarios del propio tenant.
+    
+    Restricciones:
+    - Solo ADMIN puede eliminar usuarios
+    - No se puede eliminar a sí mismo
+    - No se puede eliminar el último ADMIN del tenant
+    """
+    # Verificar permisos - solo ADMIN puede eliminar
+    if current_user.role != UserRole.ADMIN:
+        raise HTTPException(
+            status_code=status.HTTP_403_FORBIDDEN,
+            detail="Only admins can delete users"
+        )
+    
+    # No se puede eliminar a sí mismo
+    if user_id == current_user.id:
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail="You cannot delete yourself"
+        )
+    
+    # Buscar usuario
+    query = select(User).where(
+        User.id == user_id,
+        User.tenant_id == current_user.tenant_id
+    )
+    result = await db.execute(query)
+    db_user = result.scalar_one_or_none()
+    
+    if not db_user:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="User not found"
+        )
+    
+    # Verificar que no sea el último admin del tenant
+    if db_user.role == UserRole.ADMIN:
+        admin_query = select(User).where(
+            User.tenant_id == current_user.tenant_id,
+            User.role == UserRole.ADMIN,
+            User.is_active == True,
+            User.id != user_id
+        )
+        admin_result = await db.execute(admin_query)
+        active_admins = admin_result.scalars().all()
+        
+        if len(active_admins) == 0:
+            raise HTTPException(
+                status_code=status.HTTP_400_BAD_REQUEST,
+                detail="Cannot delete the last active admin of the tenant"
+            )
+    
+    # Soft delete
+    db_user.is_active = False
+    await db.commit()
+    return None
+
+
+@router.patch("/{user_id}/activate", response_model=UserResponse)
+async def activate_user(
+    user_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+    current_user: User = Depends(deps.get_current_user)
+):
+    """
+    Reactivar usuario desactivado.
+    
+    ✅ Implementa multi-tenancy: solo permite reactivar usuarios del propio tenant.
+    """
+    # Verificar permisos
+    if not current_user.can_manage_users:
+        raise HTTPException(
+            status_code=status.HTTP_403_FORBIDDEN,
+            detail="You don't have permission to activate users"
+        )
+    
+    # Buscar usuario
+    query = select(User).where(
+        User.id == user_id,
+        User.tenant_id == current_user.tenant_id
+    )
+    result = await db.execute(query)
+    db_user = result.scalar_one_or_none()
+    
+    if not db_user:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail="User not found"
+        )
+    
+    db_user.is_active = True
+    await db.commit()
+    await db.refresh(db_user)
+    return db_user

backend/app/api/v1/router.py

@@ -5,7 +5,8 @@ Router principal para la API v1
 """
 
 from fastapi import APIRouter
-from app.api.v1.endpoints import auth, health, tenants, users, systems, categories
+
+from app.api.v1.endpoints import auth, health, tenants, users, systems, categories, tickets
 
 api_router = APIRouter()
 
@@ -45,3 +46,14 @@ api_router.include_router(
     prefix="/categories",
     tags=["categories"]
 )
+
+# Tickets routes
+api_router.include_router(
+    tickets.router,
+    prefix="/tickets",
+    tags=["tickets"]
+)
+
+# Ensure FastAPI is installed in the environment
+# If not, install it using:
+# pip install fastapi

backend/app/models/category.py

@@ -1,8 +1,8 @@
-
 """
 Category Model - ServiceManagerWeb
+Categorías de tickets por tenant
 """
-from sqlalchemy import String, Text, Boolean, ForeignKey
+from sqlalchemy import String, Text, Boolean, Integer, ForeignKey, UniqueConstraint
 from sqlalchemy.orm import Mapped, mapped_column, relationship
 from sqlalchemy.dialects.postgresql import UUID
 from typing import List, Optional
@@ -11,18 +11,40 @@ import uuid
 from app.core.database import Base
 
 class Category(Base):
-    __tablename__ = "categories"
+    """Modelo de categorías de tickets (ticket_categories en BD)"""
+    __tablename__ = "ticket_categories"  # ✅ CORREGIDO: nombre correcto de tabla
     
+    # Campos básicos
     name: Mapped[str] = mapped_column(String(100), nullable=False)
-    description: Mapped[Optional[str]] = mapped_column(Text)
+    description: Mapped[Optional[str]] = mapped_column(Text, nullable=True)
-    is_active: Mapped[bool] = mapped_column(Boolean, default=True)
+    is_active: Mapped[bool] = mapped_column(Boolean, default=True, nullable=False)
     
-    # Optional: Tenant specific categories?
+    # ✅ CORREGIDO: tenant_id es obligatorio para multi-tenancy
-    tenant_id: Mapped[Optional[uuid.UUID]] = mapped_column(UUID(as_uuid=True), ForeignKey("tenants.id", ondelete="CASCADE"), nullable=True)
+    tenant_id: Mapped[uuid.UUID] = mapped_column(
+        UUID(as_uuid=True), 
+        ForeignKey("tenants.id", ondelete="CASCADE"), 
+        nullable=False  # ✅ Obligatorio
+    )
+    
+    # ✅ AÑADIDOS: Campos de SLA según schema.sql
+    color: Mapped[Optional[str]] = mapped_column(String(7), nullable=True)
+    sla_response_hours: Mapped[int] = mapped_column(Integer, default=24, nullable=False)
+    sla_resolution_hours: Mapped[int] = mapped_column(Integer, default=72, nullable=False)
+    auto_assign_to: Mapped[Optional[uuid.UUID]] = mapped_column(
+        UUID(as_uuid=True), 
+        ForeignKey("users.id"),
+        nullable=True
+    )
     
     # Relationships
     tickets: Mapped[List["Ticket"]] = relationship("Ticket", back_populates="category")
-    tenant: Mapped["Tenant"] = relationship("Tenant") # Assuming Tenant model is imported
+    tenant: Mapped["Tenant"] = relationship("Tenant")
+    auto_assign_user: Mapped[Optional["User"]] = relationship("User", foreign_keys=[auto_assign_to])
+    
+    # ✅ AÑADIDO: Constraint único por tenant (no puede haber categorías duplicadas en el mismo tenant)
+    __table_args__ = (
+        UniqueConstraint('tenant_id', 'name', name='uq_ticket_categories_tenant_name'),
+    )
 
     def __repr__(self) -> str:
-        return f"<Category(id={self.id}, name='{self.name}')>"
+        return f"<Category(id={self.id}, name='{self.name}', tenant_id={self.tenant_id})>"

backend/app/models/system.py

@@ -1,25 +1,43 @@
-
 """
 System Model - ServiceManagerWeb
+Sistemas afectados por tenant
 """
-from sqlalchemy import String, Text, Boolean
+from sqlalchemy import String, Text, Boolean, ForeignKey, UniqueConstraint
 from sqlalchemy.orm import Mapped, mapped_column, relationship
+from sqlalchemy.dialects.postgresql import UUID
 from typing import List, Optional
 import uuid
 
 from app.core.database import Base
 
 class System(Base):
-    __tablename__ = "systems"
+    """Modelo de sistemas afectados (affected_systems en BD)"""
+    __tablename__ = "affected_systems"  # ✅ CORREGIDO: nombre correcto de tabla
     
+    # Campos básicos
     name: Mapped[str] = mapped_column(String(100), nullable=False)
-    description: Mapped[Optional[str]] = mapped_column(Text)
+    description: Mapped[Optional[str]] = mapped_column(Text, nullable=True)
-    is_active: Mapped[bool] = mapped_column(Boolean, default=True)
+    is_active: Mapped[bool] = mapped_column(Boolean, default=True, nullable=False)
+    
+    # ✅ AÑADIDO: tenant_id obligatorio para multi-tenancy (faltaba completamente)
+    tenant_id: Mapped[uuid.UUID] = mapped_column(
+        UUID(as_uuid=True), 
+        ForeignKey("tenants.id", ondelete="CASCADE"), 
+        nullable=False
+    )
     
     # Relationships
-    # If we want tickets to link to systems, we will add relationship in Ticket later or now.
+    # ✅ ACTUALIZADO: nombre de relación a affected_system
-    # We will assume Ticket links to System.
+    tickets: Mapped[List["Ticket"]] = relationship(
-    tickets: Mapped[List["Ticket"]] = relationship("Ticket", back_populates="system")
+        "Ticket", 
+        back_populates="affected_system"  # ✅ Nombre actualizado
+    )
+    tenant: Mapped["Tenant"] = relationship("Tenant")
+    
+    # ✅ AÑADIDO: Constraint único por tenant (no puede haber sistemas duplicados en el mismo tenant)
+    __table_args__ = (
+        UniqueConstraint('tenant_id', 'name', name='uq_affected_systems_tenant_name'),
+    )
 
     def __repr__(self) -> str:
-        return f"<System(id={self.id}, name='{self.name}')>"
+        return f"<System(id={self.id}, name='{self.name}', tenant_id={self.tenant_id})>"

backend/app/models/tenant.py

@@ -1,9 +1,7 @@
 """
 Tenant Model - ServiceManagerWeb
-
 Modelo para organizaciones cliente (multi-tenancy)
 """
-
 from sqlalchemy import String, Integer, Text, Boolean, ARRAY
 from sqlalchemy.orm import Mapped, mapped_column, relationship
 from sqlalchemy.dialects.postgresql import UUID, ENUM
@@ -13,17 +11,14 @@ import uuid
 
 from app.core.database import Base
 
-
 class TenantStatus(str, enum.Enum):
     """Estados de un tenant."""
     ACTIVE = "active"
     SUSPENDED = "suspended"
     INACTIVE = "inactive"
 
-
 class Tenant(Base):
     """Modelo de Tenant (Organización cliente)."""
-    
     __tablename__ = "tenants"
     
     # Información básica
@@ -51,18 +46,14 @@ class Tenant(Base):
     
     # Estado
     status: Mapped[TenantStatus] = mapped_column(
-        String(20), 
+        String(20),
         default=TenantStatus.ACTIVE
     )
     
     # Relaciones
     users: Mapped[List["User"]] = relationship("User", back_populates="tenant")
     tickets: Mapped[List["Ticket"]] = relationship("Ticket", back_populates="tenant")
+    categories: Mapped[List["Category"]] = relationship("Category", back_populates="tenant")  # ✅ CORREGIDO: Era "TicketCategory"
     
     def __repr__(self) -> str:
         return f"<Tenant(id={self.id}, name='{self.name}', slug='{self.slug}')>"
-    
-    @property
-    def is_active(self) -> bool:
-        """Check if tenant is active."""
-        return self.status == TenantStatus.ACTIVE

backend/app/models/ticket.py

@@ -1,56 +1,112 @@
 """
 Ticket Model - ServiceManagerWeb
+Tickets de soporte - Core del negocio
 """
-from sqlalchemy import String, ForeignKey, Text
+from sqlalchemy import String, ForeignKey, Text, Integer, CheckConstraint, UniqueConstraint
 from sqlalchemy.orm import Mapped, mapped_column, relationship
 from sqlalchemy.dialects.postgresql import UUID, ENUM
 from typing import Optional
+from datetime import datetime
 import enum
 import uuid
 
 from app.core.database import Base
 
 class TicketStatus(str, enum.Enum):
+    """Estados posibles de un ticket"""
     NEW = "NEW"
     TRIAGE = "TRIAGE"
     IN_PROGRESS = "IN_PROGRESS"
-    WAITING_FOR_CLIENT = "WAITING_FOR_CLIENT"
+    WAITING_CUSTOMER = "WAITING_CUSTOMER"  # ✅ CORREGIDO: nombre según schema.sql
     RESOLVED = "RESOLVED"
     CLOSED = "CLOSED"
     REOPENED = "REOPENED"
 
 class TicketPriority(str, enum.Enum):
+    """Prioridades posibles de un ticket"""
     LOW = "LOW"
     MEDIUM = "MEDIUM"
     HIGH = "HIGH"
     URGENT = "URGENT"
 
 class Ticket(Base):
+    """Modelo de tickets de soporte"""
     __tablename__ = "tickets"
     
-    # Note: id, created_at, updated_at are inherited from Base
+    # Multi-tenancy
-    
+    tenant_id: Mapped[uuid.UUID] = mapped_column(
-    tenant_id: Mapped[uuid.UUID] = mapped_column(UUID(as_uuid=True), ForeignKey("tenants.id", ondelete="CASCADE"), nullable=False)
+        UUID(as_uuid=True), 
+        ForeignKey("tenants.id", ondelete="CASCADE"), 
+        nullable=False
+    )
     
+    # Campos básicos
     ticket_number: Mapped[str] = mapped_column(String(20), nullable=False)
     subject: Mapped[str] = mapped_column(String(255), nullable=False)
     description: Mapped[str] = mapped_column(Text, nullable=False)
     
-    status: Mapped[TicketStatus] = mapped_column(ENUM(TicketStatus, name="ticket_status_enum", create_type=False), default=TicketStatus.NEW)
+    # Estado y Prioridad
-    priority: Mapped[TicketPriority] = mapped_column(ENUM(TicketPriority, name="ticket_priority_enum", create_type=False), default=TicketPriority.MEDIUM)
+    status: Mapped[TicketStatus] = mapped_column(
+        ENUM(TicketStatus, name="ticket_status_enum", create_type=False), 
+        default=TicketStatus.NEW,
+        nullable=False
+    )
+    priority: Mapped[TicketPriority] = mapped_column(
+        ENUM(TicketPriority, name="ticket_priority_enum", create_type=False), 
+        default=TicketPriority.MEDIUM,
+        nullable=False
+    )
     
-    # Foreign Keys
+    # ✅ CORREGIDO: Foreign Keys apuntan a tablas correctas
-    created_by: Mapped[uuid.UUID] = mapped_column(UUID(as_uuid=True), ForeignKey("users.id"), nullable=False)
+    created_by: Mapped[uuid.UUID] = mapped_column(
-    assigned_to: Mapped[Optional[uuid.UUID]] = mapped_column(UUID(as_uuid=True), ForeignKey("users.id"), nullable=True)
+        UUID(as_uuid=True), 
+        ForeignKey("users.id"), 
+        nullable=False
+    )
+    assigned_to: Mapped[Optional[uuid.UUID]] = mapped_column(
+        UUID(as_uuid=True), 
+        ForeignKey("users.id"), 
+        nullable=True
+    )
     
-    system_id: Mapped[Optional[uuid.UUID]] = mapped_column(UUID(as_uuid=True), ForeignKey("systems.id"), nullable=True)
+    # ✅ CORREGIDO: Renombrado de system_id a affected_system_id
-    category_id: Mapped[Optional[uuid.UUID]] = mapped_column(UUID(as_uuid=True), ForeignKey("categories.id"), nullable=True)
+    affected_system_id: Mapped[Optional[uuid.UUID]] = mapped_column(
+        UUID(as_uuid=True), 
+        ForeignKey("affected_systems.id"),  # ✅ Tabla correcta
+        nullable=True
+    )
+    
+    # ✅ CORREGIDO: Foreign key a tabla correcta
+    category_id: Mapped[Optional[uuid.UUID]] = mapped_column(
+        UUID(as_uuid=True), 
+        ForeignKey("ticket_categories.id"),  # ✅ Tabla correcta
+        nullable=True
+    )
+    
+    # ✅ AÑADIDOS: Campos de SLA según schema.sql
+    sla_response_due: Mapped[Optional[datetime]] = mapped_column(nullable=True)
+    sla_resolution_due: Mapped[Optional[datetime]] = mapped_column(nullable=True)
+    first_response_at: Mapped[Optional[datetime]] = mapped_column(nullable=True)
+    resolved_at: Mapped[Optional[datetime]] = mapped_column(nullable=True)
+    
+    # ✅ AÑADIDOS: Campos de CSAT (Customer Satisfaction) según schema.sql
+    rating: Mapped[Optional[int]] = mapped_column(Integer, nullable=True)
+    rating_comment: Mapped[Optional[str]] = mapped_column(Text, nullable=True)
+    rated_at: Mapped[Optional[datetime]] = mapped_column(nullable=True)
 
     # Relationships
     tenant: Mapped["Tenant"] = relationship("Tenant", back_populates="tickets")
     
-    system: Mapped["System"] = relationship("System", back_populates="tickets")
+    # ✅ ACTUALIZADO: Nombre de relación y optional
-    category: Mapped["Category"] = relationship("Category", back_populates="tickets")
+    affected_system: Mapped[Optional["System"]] = relationship(
+        "System", 
+        back_populates="tickets"
+    )
+    
+    category: Mapped[Optional["Category"]] = relationship(
+        "Category", 
+        back_populates="tickets"
+    )
     
     created_by_user: Mapped["User"] = relationship(
         "User", 
@@ -63,3 +119,12 @@ class Ticket(Base):
         foreign_keys=[assigned_to], 
         back_populates="assigned_tickets"
     )
+    
+    # ✅ AÑADIDOS: Constraints según schema.sql
+    __table_args__ = (
+        UniqueConstraint('tenant_id', 'ticket_number', name='uq_tickets_tenant_number'),
+        CheckConstraint('rating >= 1 AND rating <= 5', name='check_rating_range'),
+    )
+
+    def __repr__(self) -> str:
+        return f"<Ticket(id={self.id}, number='{self.ticket_number}', status={self.status})>"

backend/backend/migrations/README

@@ -0,0 +1 @@
+Generic single-database configuration.

backend/backend/migrations/env.py

@@ -0,0 +1,78 @@
+from logging.config import fileConfig
+
+from sqlalchemy import engine_from_config
+from sqlalchemy import pool
+
+from alembic import context
+
+# this is the Alembic Config object, which provides
+# access to the values within the .ini file in use.
+config = context.config
+
+# Interpret the config file for Python logging.
+# This line sets up loggers basically.
+if config.config_file_name is not None:
+    fileConfig(config.config_file_name)
+
+# add your model's MetaData object here
+# for 'autogenerate' support
+# from myapp import mymodel
+# target_metadata = mymodel.Base.metadata
+target_metadata = None
+
+# other values from the config, defined by the needs of env.py,
+# can be acquired:
+# my_important_option = config.get_main_option("my_important_option")
+# ... etc.
+
+
+def run_migrations_offline() -> None:
+    """Run migrations in 'offline' mode.
+
+    This configures the context with just a URL
+    and not an Engine, though an Engine is acceptable
+    here as well.  By skipping the Engine creation
+    we don't even need a DBAPI to be available.
+
+    Calls to context.execute() here emit the given string to the
+    script output.
+
+    """
+    url = config.get_main_option("sqlalchemy.url")
+    context.configure(
+        url=url,
+        target_metadata=target_metadata,
+        literal_binds=True,
+        dialect_opts={"paramstyle": "named"},
+    )
+
+    with context.begin_transaction():
+        context.run_migrations()
+
+
+def run_migrations_online() -> None:
+    """Run migrations in 'online' mode.
+
+    In this scenario we need to create an Engine
+    and associate a connection with the context.
+
+    """
+    connectable = engine_from_config(
+        config.get_section(config.config_ini_section, {}),
+        prefix="sqlalchemy.",
+        poolclass=pool.NullPool,
+    )
+
+    with connectable.connect() as connection:
+        context.configure(
+            connection=connection, target_metadata=target_metadata
+        )
+
+        with context.begin_transaction():
+            context.run_migrations()
+
+
+if context.is_offline_mode():
+    run_migrations_offline()
+else:
+    run_migrations_online()

backend/backend/migrations/script.py.mako

@@ -0,0 +1,26 @@
+"""${message}
+
+Revision ID: ${up_revision}
+Revises: ${down_revision | comma,n}
+Create Date: ${create_date}
+
+"""
+from typing import Sequence, Union
+
+from alembic import op
+import sqlalchemy as sa
+${imports if imports else ""}
+
+# revision identifiers, used by Alembic.
+revision: str = ${repr(up_revision)}
+down_revision: Union[str, None] = ${repr(down_revision)}
+branch_labels: Union[str, Sequence[str], None] = ${repr(branch_labels)}
+depends_on: Union[str, Sequence[str], None] = ${repr(depends_on)}
+
+
+def upgrade() -> None:
+    ${upgrades if upgrades else "pass"}
+
+
+def downgrade() -> None:
+    ${downgrades if downgrades else "pass"}

backend/fix_password_script.py

@@ -1,5 +1,4 @@
-
+import asyncio
-import asyncio
 import sys
 import os
 
@@ -8,8 +7,10 @@ sys.path.append(os.path.dirname(os.path.abspath(__file__)))
 
 from sqlalchemy import select
 from app.core.database import AsyncSessionLocal
-from app.models.tenant import Tenant  # Import Tenant to register it
+from app.models.tenant import Tenant
-from app.models.ticket import Ticket  # Import Ticket to register it
+from app.models.ticket import Ticket
+from app.models.category import Category  # ✅ AÑADIR ESTO
+from app.models.system import System      # ✅ AÑADIR ESTO
 from app.models.user import User
 from app.core.security import SecurityUtils
 

backend/migrations/env.py

@@ -0,0 +1,68 @@
+from logging.config import fileConfig
+import os
+from sqlalchemy import create_engine, pool
+from sqlalchemy.engine import engine_from_config
+from alembic import context
+
+# Import Base and all models
+from app.core.database import Base
+from app.models import tenant  # Import all models explicitly
+
+# Alembic Config object
+config = context.config
+
+# Logging configuration
+if config.config_file_name:
+    fileConfig(config.config_file_name)
+
+# Get DATABASE_URL and convert to synchronous
+DATABASE_URL = os.getenv("DATABASE_URL")
+if not DATABASE_URL:
+    raise RuntimeError("DATABASE_URL environment variable is not set")
+
+SYNC_DATABASE_URL = DATABASE_URL.replace("+asyncpg", "")
+
+# Metadata for autogenerate
+target_metadata = Base.metadata
+
+
+def run_migrations_offline():
+    """
+    Run migrations in 'offline' mode.
+    """
+    context.configure(
+        url=SYNC_DATABASE_URL,
+        target_metadata=target_metadata,
+        literal_binds=True,
+        dialect_opts={"paramstyle": "named"},
+    )
+
+    with context.begin_transaction():
+        context.run_migrations()
+
+
+def run_migrations_online():
+    """
+    Run migrations in 'online' mode.
+    """
+    # Fetch the URL from Alembic configuration
+    alembic_config = config.get_section(config.config_ini_section)
+    alembic_config["sqlalchemy.url"] = SYNC_DATABASE_URL
+
+    connectable = engine_from_config(
+        alembic_config,
+        prefix="sqlalchemy.",
+        poolclass=pool.NullPool,
+    )
+
+    with connectable.connect() as connection:
+        context.configure(connection=connection, target_metadata=target_metadata)
+
+        with context.begin_transaction():
+            context.run_migrations()
+
+
+if context.is_offline_mode():
+    run_migrations_offline()
+else:
+    run_migrations_online()

fix_admin_password.py

@@ -0,0 +1,67 @@
+"""
+Script para actualizar el password del usuario admin
+Ejecutar: python fix_admin_password.py
+"""
+import asyncio
+import sys
+from sqlalchemy import select, update
+from passlib.context import CryptContext
+
+# Importar desde el proyecto
+sys.path.insert(0, '/app')
+from app.core.database import AsyncSessionLocal
+from app.models.user import User
+
+# Configurar passlib igual que en security.py
+pwd_context = CryptContext(
+    schemes=["argon2", "bcrypt"],
+    deprecated="auto",
+    argon2__memory_cost=65536,
+    argon2__time_cost=3,
+    argon2__parallelism=4,
+)
+
+async def fix_admin_password():
+    """Actualizar password del admin a 'admin123'"""
+    
+    # Generar hash del password
+    new_password = "admin123"
+    password_hash = pwd_context.hash(new_password)
+    
+    print(f"Nuevo hash generado para password: {new_password}")
+    print(f"Hash: {password_hash[:50]}...")
+    
+    async with AsyncSessionLocal() as session:
+        try:
+            # Buscar usuario admin
+            result = await session.execute(
+                select(User).where(User.email == "admin@aduanasoft.com")
+            )
+            user = result.scalar_one_or_none()
+            
+            if not user:
+                print("❌ Usuario admin no encontrado")
+                return
+            
+            print(f"✅ Usuario encontrado: {user.email} (ID: {user.id})")
+            
+            # Actualizar password
+            user.password_hash = password_hash
+            
+            await session.commit()
+            
+            print("✅ Password actualizado exitosamente")
+            print(f"   Email: admin@aduanasoft.com")
+            print(f"   Password: admin123")
+            
+        except Exception as e:
+            await session.rollback()
+            print(f"❌ Error: {e}")
+            raise
+
+if __name__ == "__main__":
+    print("=" * 60)
+    print("ACTUALIZAR PASSWORD DEL ADMIN")
+    print("=" * 60)
+    asyncio.run(fix_admin_password())
+    print("=" * 60)

frontend-client/src/lib/components/Header.svelte

@@ -59,6 +59,9 @@
             <button
               on:click={toggleMenu}
               class="flex items-center space-x-2 text-gray-700 hover:text-primary-600 focus:outline-none focus:ring-2 focus:ring-primary-500 focus:ring-offset-2 rounded-md p-2"
+              role="button"
+              tabindex="0"
+              on:keydown={(e) => e.key === 'Enter' && toggleMenu()}
             >
               <div class="w-8 h-8 bg-primary-100 rounded-full flex items-center justify-center">
                 <span class="text-primary-600 text-sm font-medium">
@@ -87,6 +90,9 @@
                   <button
                     on:click={handleLogout}
                     class="block w-full text-left px-4 py-2 text-sm text-gray-700 hover:bg-gray-100"
+                    role="button"
+                    tabindex="0"
+                    on:keydown={(e) => e.key === 'Enter' && handleLogout()}
                   >
                     Cerrar Sesión
                   </button>

frontend-client/src/lib/stores/tickets.ts

@@ -1,7 +1,6 @@
-import { writable } from 'svelte/store';
+import { writable, get } from 'svelte/store';
-import { auth } from './auth.js';
-import { get } from 'svelte/store';
 import type { Writable } from 'svelte/store';
+import { auth } from './auth';
 
 // Types
 export interface Ticket {
@@ -84,8 +83,29 @@ async function apiCall(endpoint: string, options: RequestInit = {}) {
   });
 
   if (!response.ok) {
-    const error = await response.json();
+    let errorMessage = 'Request failed';
-    throw new Error(error.detail || 'Request failed');
+    try {
+      const error = await response.json();
+      console.error('❌ API Error Response:', error);
+      
+      // Manejar diferentes formatos de error de FastAPI
+      if (error.detail) {
+        if (Array.isArray(error.detail)) {
+          // Errores de validación de FastAPI
+          errorMessage = error.detail.map(e => `${e.loc.join('.')}: ${e.msg}`).join(', ');
+        } else if (typeof error.detail === 'string') {
+          errorMessage = error.detail;
+        } else {
+          errorMessage = JSON.stringify(error.detail);
+        }
+      } else {
+        errorMessage = JSON.stringify(error);
+      }
+    } catch (e) {
+      errorMessage = `HTTP ${response.status}: ${response.statusText}`;
+    }
+    
+    throw new Error(errorMessage);
   }
 
   return response.json();
@@ -100,13 +120,13 @@ function createTicketsStore() {
 
     // Load user's tickets
     loadTickets: async () => {
-      update(state => ({ ...state, isLoading: true, error: null }));
+      update((state: TicketsState) => ({ ...state, isLoading: true, error: null }));
       
       try {
         const tickets = await apiCall('/tickets/');
-        update(state => ({ ...state, tickets, isLoading: false }));
+        update((state: TicketsState) => ({ ...state, tickets, isLoading: false }));
       } catch (error) {
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           isLoading: false, 
           error: error instanceof Error ? error.message : 'Failed to load tickets' 
@@ -116,7 +136,7 @@ function createTicketsStore() {
 
     // Load specific ticket with details
     loadTicket: async (ticketId: string) => {
-      update(state => ({ ...state, isLoading: true, error: null }));
+      update((state: TicketsState) => ({ ...state, isLoading: true, error: null }));
       
       try {
         const [ticket, comments, attachments] = await Promise.all([
@@ -125,7 +145,7 @@ function createTicketsStore() {
           apiCall(`/tickets/${ticketId}/attachments`)
         ]);
 
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           currentTicket: ticket, 
           comments, 
@@ -133,7 +153,7 @@ function createTicketsStore() {
           isLoading: false 
         }));
       } catch (error) {
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           isLoading: false, 
           error: error instanceof Error ? error.message : 'Failed to load ticket' 
@@ -143,15 +163,27 @@ function createTicketsStore() {
 
     // Create new ticket
     createTicket: async (ticket: CreateTicketRequest) => {
-      update(state => ({ ...state, isLoading: true, error: null }));
+      update((state: TicketsState) => ({ ...state, isLoading: true, error: null }));
       
       try {
+        // Mapear campos del frontend al formato del backend
+    const ticketData = {
+      subject: ticket.title,           // ← Backend espera "subject" no "title"
+      description: ticket.description,
+      category_id: ticket.category_id,
+      priority: ticket.priority,
+      system_id: null                  // ← Opcional
+      };
+    
+
+      console.log('Sending ticket data:', ticketData);
+
         const newTicket = await apiCall('/tickets/', {
           method: 'POST',
-          body: JSON.stringify(ticket)
+          body: JSON.stringify(ticketData)
         });
 
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           tickets: [newTicket, ...state.tickets], 
           isLoading: false 
@@ -159,7 +191,8 @@ function createTicketsStore() {
 
         return newTicket;
       } catch (error) {
-        update(state => ({ 
+        console.error('Create ticket error:', error);
+        update((state: TicketsState) => ({ 
           ...state, 
           isLoading: false, 
           error: error instanceof Error ? error.message : 'Failed to create ticket' 
@@ -176,14 +209,14 @@ function createTicketsStore() {
           body: JSON.stringify({ content })
         });
 
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           comments: [...state.comments, comment] 
         }));
 
         return comment;
       } catch (error) {
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           error: error instanceof Error ? error.message : 'Failed to add comment' 
         }));
@@ -213,14 +246,14 @@ function createTicketsStore() {
 
         const attachment = await response.json();
         
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           attachments: [...state.attachments, attachment] 
         }));
 
         return attachment;
       } catch (error) {
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           error: error instanceof Error ? error.message : 'Failed to upload attachment' 
         }));
@@ -236,15 +269,15 @@ function createTicketsStore() {
           body: JSON.stringify({ resolution })
         });
 
-        update(state => ({
+        update((state: TicketsState) => ({
           ...state,
           currentTicket: state.currentTicket?.id === ticketId ? updatedTicket : state.currentTicket,
-          tickets: state.tickets.map(t => t.id === ticketId ? updatedTicket : t)
+          tickets: state.tickets.map((t: Ticket) => t.id === ticketId ? updatedTicket : t)
         }));
 
         return updatedTicket;
       } catch (error) {
-        update(state => ({ 
+        update((state: TicketsState) => ({ 
           ...state, 
           error: error instanceof Error ? error.message : 'Failed to close ticket' 
         }));
@@ -254,12 +287,12 @@ function createTicketsStore() {
 
     // Clear error
     clearError: () => {
-      update(state => ({ ...state, error: null }));
+      update((state: TicketsState) => ({ ...state, error: null }));
     },
 
     // Clear current ticket
     clearCurrentTicket: () => {
-      update(state => ({ 
+      update((state: TicketsState) => ({ 
         ...state, 
         currentTicket: null, 
         comments: [], 

frontend-internal/src/routes/tenants/+page.svelte

@@ -43,11 +43,15 @@
   async function handleSubmit() {
     try {
       if (editingTenant) {
-        await api.put(`/tenants/${editingTenant.id}`, formData);
+        // Asegurarse de enviar el campo "status" correctamente
-        toast.success('Cliente actualizado');
+        const updatedData = { ...formData, status: formData.is_active ? 'active' : 'inactive' };
+        await api.put(`/tenants/${editingTenant.id}`, updatedData);
+        toast.success(`Cliente ${formData.is_active ? 'activado' : 'desactivado'} correctamente`);
       } else {
-        await api.post('/tenants/', formData);
+        // Asegurarse de enviar el campo "status" al crear un cliente
-        toast.success('Cliente creado');
+        const newData = { ...formData, status: formData.is_active ? 'active' : 'inactive' };
+        await api.post('/tenants/', newData);
+        toast.success('Cliente creado correctamente');
       }
       showModal = false;
       loadTenants();
@@ -152,6 +156,11 @@
       <button type="button" on:click={() => showModal = false} class="mt-3 w-full inline-flex justify-center rounded-md border border-gray-300 shadow-sm px-4 py-2 bg-white text-base font-medium text-gray-700 hover:bg-gray-50 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-indigo-500 sm:mt-0 sm:col-start-1 sm:text-sm">
         Cancelar
       </button>
+      {#if editingTenant}
+        <button type="button" on:click={async () => { await api.delete(`/tenants/${editingTenant.id}`); toast.success('Cliente eliminado'); showModal = false; loadTenants(); }} class="w-full inline-flex justify-center rounded-md border border-transparent shadow-sm px-4 py-2 bg-red-600 text-base font-medium text-white hover:bg-red-700 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-red-500 sm:col-start-1 sm:text-sm">
+          Eliminar
+        </button>
+      {/if}
     </div>
   </form>
 </Modal>

frontend-internal/vite.config.js

@@ -1,24 +1,24 @@
-import { sveltekit } from '@sveltejs/kit/vite';
+import { sveltekit } from '@sveltejs/kit/vite';
 import { defineConfig } from 'vite';
 
 export default defineConfig({
-	plugins: [sveltekit()],
+        plugins: [sveltekit()],
-	server: {
+        server: {
-		port: 3000,
+                port: 3000,
-		host: '0.0.0.0',
+                host: '0.0.0.0',
-		proxy: {
+                proxy: {
-			'/api/v1': {
+                        '/api': {
-				target: 'http://servicemanager-backend:8000',
+                                target: 'http://servicemanager-backend:8000',
-				changeOrigin: true,
+                                changeOrigin: true,
-				rewrite: (path) => path.replace(/^\/api/, '')
+                                rewrite: (path) => path.replace(/^\/api/, '')
-			}
+                        }
-		}
+                }
-	},
+        },
-	preview: {
+        preview: {
-		port: 3000,
+                port: 3000,
-		host: '0.0.0.0'
+                host: '0.0.0.0'
-	},
+        },
-	build: {
+        build: {
-		target: 'esnext'
+                target: 'esnext'
-	}
+        }
 });