Skip to content

Change/update talks info#11

Closed
LiliGC wants to merge 9 commits into
mainfrom
change/update-talks-info
Closed

Change/update talks info#11
LiliGC wants to merge 9 commits into
mainfrom
change/update-talks-info

Conversation

@LiliGC
Copy link
Copy Markdown
Member

@LiliGC LiliGC commented Jun 1, 2025

🚀 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

  • Datos completos de charlas y speakers en data/talks.js.
  • Agregadas imágenes de speakers en formato WebP para mejor rendimiento en images/speakers/*webp.
  • Eliminadas 9 imágenes placeholder obsoletas de speakers.
  • Agenda visible en /santiago, /valparaiso y /talks.
  • Botón “Proponer una charla” no se muestra en main, se eliminó el proposallink de santiago y valparaiso en data cities y ahora se muestra un botón que lleva a la agenda.
  • Mensaje “Cupos agotados” en Santiago y no el botón de registro cuando el regisrationstatus is soldout.
  • Soporte para múltiples speakers y perfiles LinkedIn en TalkCard.
  • Estado “agotado” implementado en registro con RegistrationState y FeatureGuard.
  • Bordes y badges de colores para tipos de eventos.
  • Visualización completa de descripciones de charlas con sus tags.

🧪 Lista de Verificación para Testing

  • Todas las imágenes de speakers se muestran correctamente.
  • Múltiples speakers se muestran apropiadamente en las tarjetas de charlas.
  • Los enlaces de LinkedIn son funcionales y se abren en nuevas pestañas.
  • Los bordes con códigos de color y badges se muestran correctamente.
  • El estado de registro muestra "agotado" cuando corresponde.
  • Las descripciones de charlas se muestran completas sin truncar.
  • El diseño responsivo funciona en móvil y escritorio.
  • Todas las páginas cargan sin errores en consola.

📷 Imágenes

Main page (General)

imagen

Registro Ciudad Santiago

imagen

Agenda Ciudad Santiago múltiples speakers

imagen

Agenda Ciudad Valparaíso (Charla, taller en el mismo horario)

imagen

Página de Charlas (General)

imagen


Closes #10

Copy link
Copy Markdown
Member

@Tony-Rome Tony-Rome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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:
    image

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:
image


  • Redundancia perfil LinkedIn
    Se muestra el logo y el nombre como tal de LinkdIn:
    image

  • 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:
    image

  • 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:
    image

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!

@Tony-Rome Tony-Rome added the enhancement New feature or request label Jun 2, 2025
Copy link
Copy Markdown
Member

@pablolirag pablolirag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muchas gracias @LiliGC , revise lo mas que pude, a mi parecer esta todo correcto.

@LiliGC
Copy link
Copy Markdown
Member Author

LiliGC commented Jun 2, 2025

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.

@Tony-Rome
Copy link
Copy Markdown
Member

Tony-Rome commented Jun 2, 2025

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.

@LiliGC LiliGC closed this Jun 2, 2025
@LiliGC LiliGC deleted the change/update-talks-info branch June 2, 2025 02:24
@LiliGC LiliGC mentioned this pull request Jun 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Agregar agenda para PyDay Santiago y PyDay Valparaíso y manejo de registro

3 participants