Skip to content

Fix: Verificacion de instituciones#104

Draft
TomasAlvarez0 wants to merge 3 commits into
mainfrom
fix/verificacion-cta-y-gestion-documentos
Draft

Fix: Verificacion de instituciones#104
TomasAlvarez0 wants to merge 3 commits into
mainfrom
fix/verificacion-cta-y-gestion-documentos

Conversation

@TomasAlvarez0

@TomasAlvarez0 TomasAlvarez0 commented Mar 30, 2026

Copy link
Copy Markdown
Collaborator

Permite reintentar la verificacion en caso de subir archivo equivocado, antes directamente se inhabilitaba el usuario y ya no podia acceder. Ademas se agregan accesos a la gestion de la verificacion por el panel y por el badge del perfil.

🚀 Pull Request – Conectando Corazones

Alinea los estados de verificación institucional para diferenciar omitido, en revisión y rechazado en dashboard/perfil.

  • Agrega CTAs accionables coherentes (Verificar, Gestionar verificación, Reenviar verificación) y corrige el desfasaje de mensaje/acción.
  • Habilita gestión de documentación enviada en /institucion/verificacion (listar, abrir y eliminar archivos para reemplazo).

Test plan

  • Probar cuenta institución sin documentación cargada: verificar CTA Verificar en dashboard y badge de perfil.
  • Probar cuenta con verificación pendiente y documentos cargados: validar CTA Gestionar verificación y listado de archivos existentes.
  • Probar eliminación de archivo existente y posterior carga de reemplazo.
  • Probar cuenta con verificación rechazada: validar CTA Reenviar verificación y mensaje con motivo.
  • Probar cuenta aprobada: confirmar que no se muestran CTAs de verificación.-

📝 Checklist

  • Alineado al DER y contratos de datos (types, mocks, DTOs).
  • Documentación y mocks actualizados si aplica.
  • El código sigue los lineamientos generales del proyecto.
  • Pasan los tests existentes y los nuevos.
  • Estoy de acuerdo con seguir el Código de Conducta al enviar este PR.
  • He leido la Guía para Contribuir al proyecto.

🧪 Pruebas realizadas

Detallá qué hiciste para validar que funciona y no rompe nada.
Ej.: formulario de registro probado con casos válidos/invalidos.

📸 Capturas (opcional)

Captura de pantalla 2026-03-30 115701 Captura de pantalla 2026-03-30 115740 Captura de pantalla 2026-03-30 120449 Captura de pantalla 2026-03-30 120515 Captura de pantalla 2026-03-30 120623

Se unifica el comportamiento de CTAs y badges para distinguir cuentas que omitieron documentaci?n, rechazadas y en revisi?n. Tambi?n se habilita visualizar y eliminar documentos de verificaci?n ya cargados para reemplazarlos desde la misma pantalla.

Made-with: Cursor
@vercel

vercel Bot commented Mar 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
conectando-corazones Ready Ready Preview, Comment Mar 30, 2026 6:32pm

@TomasAlvarez0 TomasAlvarez0 marked this pull request as ready for review March 30, 2026 18:35
@github-actions

Copy link
Copy Markdown

⚠️ Tenés múltiples PRs activos.

Para evitar trabajo fragmentado y acelerar las revisiones, es importante cerrar lo que ya está en curso antes de abrir nuevas líneas de trabajo.

PRs activos:

👉 Prioridad sugerida: cerrar primero #105 (#105) (el más antiguo).

Mantener foco mejora el throughput del equipo y reduce tiempos de integración.

@github-actions

Copy link
Copy Markdown

Se ha configurado este PR y asignado responsabilidades claras con éxito:

¡Éxitos con la revisión de código! 🚀

@MVRU

MVRU commented Mar 30, 2026

Copy link
Copy Markdown
Owner

@TomasAlvarez0 ojo que seguís usando Svelte 4!

@github-actions github-actions Bot added the ready-for-review Pull request listo para revisión label Mar 30, 2026
@github-actions

Copy link
Copy Markdown

🤖 Este PR estaba abierto y no marcado como ready-for-review.

Se asume listo para revisión y se activa el flujo automáticamente.

@TomasAlvarez0

Copy link
Copy Markdown
Collaborator Author

Cierto, me quedaron los cambios de anoche y hoy los pegue en el proyecto levantado nuevo! lo corrijo

@MVRU MVRU marked this pull request as draft April 9, 2026 02:19
@MVRU MVRU removed ready-for-review Pull request listo para revisión needs-focus needs-review labels Apr 9, 2026
Elimina dependencias de svelte/legacy en rutas migradas y corrige contratos de datos/tipos para que el flujo de solicitar cierre y el chequeo del proyecto compilen limpio con Svelte 5.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants