Repository navigation
feat(notificaciones): campana Realtime en Navbar y servicio simulado de correo - #57
Merged
Merged
Conversation
Collaborator
Author
juanjose1234456787654
approved these changes
Nov 9, 2026
juanjose1234456787654
left a comment
Owner
There was a problem hiding this comment.
Revisado y aprobado. ✅
- Verificados los cambios del commit
c3838d2: la inclusión del evento paraPENDIENTE_APROBACIONy el trigger deenviarCorreoSimuladofuncionan según lo esperado. - Suscripción limpia a Supabase Realtime filtrando correctamente por
pasajeroId. - Componente
NotificacionCampana.tsxbien integrado en elNavbarcon conteo reactivo y acción de marcado como leída. - Plantillas en
emailTemplates.tsconsistentes con los 3 estados del flujo. - Tipado estricto en TS sin errores y sin presencia de
mockData.
Listo para merge a develop. 👍
The-Fabry
approved these changes
Nov 9, 2026
The-Fabry
left a comment
Collaborator
There was a problem hiding this comment.
Revisión y Aprobación: Campana Realtime en Navbar y servicio simulado de correo (Issue #32)
Excelente trabajo con este módulo de notificaciones. Revisé el código y las capturas de la consola que adjuntaste. El sistema funciona muy bien y es una gran mejora para mantener informado al pasajero sobre el estado de su compra.
Puntos que revisé y me gustaron:
-
Campana de Notificaciones (NotificacionCampana.tsx):
- El componente visual está muy bien hecho. Tiene el contador de notificaciones no leídas, el menú desplegable se cierra al hacer clic afuera y permite marcar las notificaciones como leídas.
- La conexión con Supabase Realtime está perfecta. Se filtra por el
pasajeroIdpara que cada usuario reciba solo sus propias notificaciones en tiempo real.
-
Servicio de Correo Simulado:
- Excelente trabajo con los tres eventos: Pago Recibido, Boleto Aprobado y Pago Rechazado.
- Las plantillas (
emailTemplates.ts) son claras y fáciles de leer, lo que simula muy bien cómo se vería un correo real.
-
Solución de Conflictos y Mejoras:
- Veo que corregiste el evento "Pago Recibido" usando el
else if, lo cual evita que se mezclen los mensajes. - La captura de la consola demuestra que el correo simulado se está enviando correctamente en el navegador.
- Veo que corregiste el evento "Pago Recibido" usando el
Revisión de la Lista de Verificación:
- Rama creada desde develop.
- Commits con el formato correcto.
- Compilación exitosa.
- Evidencia visual y de consola adjunta en el Pull Request.
- Primera aprobación recibida (Juan José).
Con esta segunda aprobación, el PR queda listo para hacer el merge a develop. ¡Muy buen trabajo!
This was referenced Nov 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Closes #32
📌 Descripcion
Implementación del módulo de notificaciones en tiempo real con Supabase Realtime. Se agregó una campana en el Navbar con contador de no leídas, un desplegable para ver las notificaciones y un servicio simulado de correo electrónico para los eventos clave del ciclo de pago.
🛠️ Detalles de la Implementación
NotificacionCampana.tsxcon badge, desplegable y opción de marcar como leída.Boletofiltrados porpasajeroId. Detecta estadosCOMPLETADO/PAGADOyRECHAZADO/CANCELADOpara mostrar la notificación.enviarCorreoSimuladoenemailService.tsque registra en consola los envíos.emailTemplates.tscon las 3 plantillas (Pago Recibido, Boleto Aprobado, Pago Rechazado).🏁 Definición de Terminado (Checklist)
any).mockData.feat(notificaciones): <descripción>.developconCloses #32(A la espera de las 2 aprobaciones).📸 Evidencia Visual