Change/update talks info#11
Conversation
…akers, LinkedIn integration and full descriptions
There was a problem hiding this comment.
Por si a caso dejé un resumen al final de este escrito
General
Los cambios funcionan bien en local (móvil - desktop) y también en el código nada raro veo.
Se cumple el propósito principal de poder mostrar la agenda según su sede. Por lo que se aprueban los cambios, solo pido por favor para avanzar con PR actualizar nombre de rama según la docu oficial para nombrar rama. Por ejemplo change-10-agenda-sedes.
Encontré unos pequeños detalles de diseño que no interrumpen con la navegación ni con el cronograma, por lo que en opinión personal es mejor que pasen con esta PR y que quede el registro en el apartado de Issues.
Adjunto detalles encontrados:
- Menú
Al estar al inicio de la vista, cuando se abre el menú este desplaza hacia abajo los elementos:

Esto pasa solamente al estar al inicio de la vista, si se abre o se sigue con el scroll, menú queda por encima de los elementos:

- Redundancia secciones para ver charlas
Se tiene dos secciones para ver y filtrar charlas. La superior tiene separado por sedes y la inferior tiene un filtro, generando cruce de datos:

- Charlas sin distinción por sede en vista general charlas
Al entrar a la vista y visualizar todas las charlas, no existe un tag (por ejemplo) para distinguir a qué sede corresponde cada charla.
Entiendo que se dividen por las distintas salas "computación" para Santiago y "sala xxxxx" para la Valparaíso, pero a simple viste no se aprecia
- Accesibilidad
Ocupé la herramienta Lighthouse para testear la página en local y me arroja problemas con accesibilidad relacionado al efecto del background:

Resumen
- PR Se aprueba, cumple con el objetivo, se pide solamente renombrar rama.
- Se encuentran detalles en diseño que no entorpecen funcionalidad ni navegación. Crearé las Issues correspondientes para que quede el registro de estos detalles para corregirlos.
Muchas gracias Lili por el desarrollo 🧙♀️ quedó excelente!
pablolirag
left a comment
There was a problem hiding this comment.
Muchas gracias @LiliGC , revise lo mas que pude, a mi parecer esta todo correcto.
|
Ya, hice la modificación de Linkedin que dijo Tony, estoy arreglando lo otro de la página de charlas, y después cambio el nombre a la rama, el issue lo creé después de que ya tenía la rama. Lo de accesibilidad con lighthouse no lo había probado. |
Lili pero no es necesario que hagas esos arreglos, no son importantes, basta con que esten los issues, no es necesario que te desgastes, lo podemos ver después en conjunto con el equipo. Respecto a la PR, se puede hacer merge igual ahora, solo pido actualizar el nombre de la rama. |

🚀 Actualización Agenda PyDay Chile 2025 (#10)
📋 Resumen
Actualiza la agenda por ciudad (Santiago y Valparaíso), gestión de registro según cupos, y mejora componentes para mejor experiencia y usabilidad.
🔧 Cambios principales
data/talks.js.images/speakers/*webp./santiago,/valparaisoy/talks.TalkCard.RegistrationStateyFeatureGuard.🧪 Lista de Verificación para Testing
📷 Imágenes
Main page (General)
Registro Ciudad Santiago
Agenda Ciudad Santiago múltiples speakers
Agenda Ciudad Valparaíso (Charla, taller en el mismo horario)
Página de Charlas (General)
Closes #10