Skip to content

fix: listado de catalogos, alineación en alta/edición y redirección de catalogos inexistentes.

juan.viera.c requested to merge feature/gestion-catalogos into dev

Descripción:

Qué resuelve

1. /catalogos daba 404 en dev. El commit que pasó cada pestaña de Catálogos a su propia ruta (adbfd96) dejó afuera la página del listado catalogos/[catalogo]/page.tsx. /catalogos redirigía a /catalogos/carreras (y los formularios volvían al listado al guardar), y todo terminaba en "Página no encontrada".

La página nueva resuelve el catálogo de la URL, exige el permiso de ver de ese catálogo (si no, muestra SinPermiso) y le pasa a CatalogosView el ?exito= que deja el formulario al guardar.

2. El alta y la edición "saltaban" al medio de la pantalla. El formulario era una columna de 600px centrada, mientras que el listado usa un contenedor de 1200px. Ahora CatalogoFormLayout (los cinco catálogos) y el aviso de error de [id]/editar usan el mismo contenedor que el listado, con la columna de 600px alineada a la izquierda.

3. Catálogo inexistente en la URL → redirige a /catalogos (decisión). Antes, /catalogos/carrera (mal escrito o link viejo) llamaba a notFound(). La página 404 es una pantalla de fuera de sesión (AuthLayout) y, disparada desde adentro de la app, se dibujaba debajo de la TopBar: logo duplicado y fondo desbordado, sobre todo en celular. Ahora el listado, el alta y la edición rediria al primer catálogo que el usuario puede ver. Las URLs que no existen en la app siguen mostrando el 404 normal, que se ve bien. Queda documentado en el README.

Archivos

  • src/app/(app)/(secciones)/catalogos/[catalogo]/page.tsx (nuevo)
  • src/app/(app)/(secciones)/catalogos/[catalogo]/nuevo/page.tsx
  • src/app/(app)/(secciones)/catalogos/[catalogo]/[id]/editar/page.tsx
  • src/components/catalogos/CatalogoFormLayout.tsx
  • README.md (sección Catálogos: rutas y la decisión del punto 3; se saca la nota "Falta commitear la página de listado")

La rama incluye un merge de dev para traer el refactor de BackButton de mejoras-generales, que tocaba CatalogoFormLayout. No hubo conflictos.

Cómo probar

  • Como admin, entrar a Catálogos desde el menú: abre Carreras con el listado; las 5 pestañas navegan.
  • Como docente, /catalogos → "No tenés permiso para ver esta página".
  • "+ Nueva carrera" (y el resto de los catálogos): el formulario queda alineado a la izquierda con el listado, sin saltar al medio.
  • Guardar un alta: vuelve al listado con el mensaje de éxito y la URL queda sin ?exito=.
  • Una URL mal escrita, como /catalogos/carrera, lleva al listado de catálogos.
  • /catalogos/carreras/999999/editar → aviso "Carrera no encontrada" alineado igual que el formulario.
Edited by Agustín Cantero

Merge request reports