Skip to content

Conversation

@MariferVL
Copy link
Contributor

Hola equipo 👋

Este PR incluye mi presentación como nueva integrante de coordinación:

  • Se agregó la imagen correspondiente en la ruta y formato solicitados:
    content/images/coordinacion_maria-fernanda_villalobos.png
  • Se creó el archivo .md en la ruta indicada y con el nombre sugerido:
    content/2025-04-18-presentacion-mariafernanda-villalobos.md
  • El contenido es personalizado e incluye algunos elementos nuevos que probé para darle un toque distintivo (por ejemplo, badges y un timeline tipo Gantt).

Sobre los elementos nuevos:

  • Badges (shields.io): ya verifiqué que se visualizan correctamente, así que no requieren acción extra.
  • Timeline tipo Gantt (Mermaid): esta parte no se renderiza localmente con pelican -l, por eso:
    • Dejé el código Mermaid comentado, tal como lo había diseñado originalmente desde el editor web de GitHub (donde visualmente quedaba muy bien).
    • Además, añadí una imagen estática del gráfico, generada a partir del mismo código, que sí se verá correctamente.

Con esto, queda a criterio de ustedes qué versión mantener: si el gráfico comentado puede renderizarse bien en staging/producción o si prefieren mantener solo la imagen.

Mi intención es aportar sin generar trabajo adicional, así que si algo no funciona bien, por favor háganmelo saber y lo simplifico sin problema.

¡Muchas gracias por su revisión y orientación! 🙏

— MariferVL

@MariferVL MariferVL requested a review from a team as a code owner April 19, 2025 03:18
Copy link
Member

@KrlitosForever KrlitosForever left a comment

Choose a reason for hiding this comment

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

Hola @MariferVL Excelente presentación 🎉 me gusto mucho ver los badges 🦾 y sobre todo tu timeline lo encontré muy original 😎. Te deje unos comentarios para que los revises.
Y con eso yo aprobaría tu PR. Excelente fin de semana👋

@MariferVL
Copy link
Contributor Author

¡Gracias por las revisiones y correcciones detalladas! 🙏

He aplicado todas las sugerencias indicadas. Respecto al emoji de la bandera 🇨🇱, lo adapté usando una imagen SVG de Twemoji (biblioteca oficial de emojis de Twitter) para garantizar compatibilidad universal:

<img 
  src="https://rawcdn.githack.com/twitter/twemoji/v14.0.2/assets/svg/1f1e8-1f1f1.svg" 
  alt="🇨🇱" 
  style="height: 1em;"
>

@KrlitosForever KrlitosForever self-requested a review April 20, 2025 13:27
Copy link
Member

@KrlitosForever KrlitosForever left a comment

Choose a reason for hiding this comment

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

Excelente!! ultra-mega-requete-aprobado 🎉

@KrlitosForever
Copy link
Member

Ahora debemos esperar 24 si nadie más revisa. Y si alguien revisa y lo aprueba puedes realizar el Merge de inmediato.

@MariferVL
Copy link
Contributor Author

Ok! Muchas gracias @KrlitosForever. 🙏🏻🤗
Entonces 24 hr y descontando. ⏲️

Copy link
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.

Desconozco lo de mermaid que funcionará en prod, en lo personal prefiero que quede el enlace de la imagen, sino subir la imagen directo a github.

La sugerencia del timeline con leyenda es solo si no te es complicado hacerlo, mi idea tampoco es generar trabajo demás, como también puedes aplicar el cambio que gustes, no es necesario seguir mi opinión.

Sobre la presentación, me gusto harto, precisa y concisa. La frase final muy buena 🧙‍♂️ muchas gracias por el trabajo!

@MariferVL
Copy link
Contributor Author

Hola @Tony-Rome!
Gracias por tomarte el tiempo para revisar mi PR y por todos tus comentarios. Te cuento cómo voy a manejar cada punto:

Imagen:
Tienes toda la razón: en la documentación se especifica una imagen de 330x330. Se me pasaron esos 30 píxeles, y aunque el resultado en 300x300 resulta nítido, actualizaré el tamaño de inmediato para cumplir con lo indicado.

Código comentado:
Voy a borrar el código comentado en la versión final para dejar el repo limpio. Solo lo dejé de forma temporal mientras esperaba que se integre la librería necesaria.

Diagrama Gantt:
Sobre el diagrama, gracias por la sugerencia de incluir una leyenda para los hitos. Como lo generé con Mermaid, seguí el estándar profesional del Gantt, y creo que las líneas verticales y el formato ya comunican bien los períodos. Además, mantendré el enlace de la imagen para asegurar que funcione correctamente en producción. Aprecio tu enfoque y lo tomo en cuenta, pero por ahora lo dejaré así.

¡Nuevamente, gracias por el feedback y los cumplidos sobre la presentación! Es genial recibir comentarios tan constructivos.

Saludos!

@MariferVL MariferVL requested a review from Tony-Rome April 21, 2025 03:25
Copy link
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.

Gracias por el trabajo! cuando gustes puedes hacer el merge.

@MariferVL MariferVL merged commit 2890852 into main Apr 21, 2025
1 check passed
@MariferVL MariferVL deleted the pch-change-154-agregar-perfil branch April 21, 2025 04:28
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.

Agregar a la página de Coordinación y generar post Maria-Fernanda Villalobos

4 participants