Skip to content

Feat: habilitar link en sponsors y ajustar orden en botones de /talks#4

Merged
MariferVL merged 4 commits intomainfrom
feat/update-sponsors-talks-buttons
May 6, 2025
Merged

Feat: habilitar link en sponsors y ajustar orden en botones de /talks#4
MariferVL merged 4 commits intomainfrom
feat/update-sponsors-talks-buttons

Conversation

@MariferVL
Copy link
Contributor

@MariferVL MariferVL commented May 5, 2025

Este PR resuelve los siguientes issues:

Closes #1
Closes #2
Closes #3

imagen

Además, aprovechéde incorporar deploy.yml.

Para que el workflow de despliegue funcione correctamente, por favor añade lo siguiente en
Settings > Secrets and variables >Actions:

Secrets:

  1. SSH_PRIVATE_KEY – clave privada SSH (contenido de gh_rsa) para autenticarte sin contraseña Stack Overflow.

  2. SSH_KNOWN_HOSTS – salida de ssh-keyscan -p ${{ secrets.VPS_PORT }} ${{ secrets.VPS_HOST }} para verificar la huella del servidor.

  3. VPS_HOST – IP o dominio de la VPS donde desplegamos.

  4. VPS_USER – usuario SSH en la VPS (por ejemplo pyday).

  5. VPS_PORT – puerto SSH (normalmente 22, o el que use el servidor).

  6. TARGET_PATH – ruta remota donde copiar los ficheros (e.g. /home/pyday).

Nota: necesitas privilegios de administrador para ver/crear estos secrets en el repositorio. Esos permisos los tiene @pablolirag y yo.

Variables:

Ya incorporé las vars no sensibles en variables de repositorio:

  • NEXT_PUBLIC_SITE_URL – URL pública del sitio.
  • NEXT_PUBLIC_FEATURE_REGISTRATION"false" o "true" para activar/desactivar registro de usuarios.
  • NEXT_PUBLIC_FEATURE_SPONSORS"false" o "true" para sección de patrocinadores.
  • NEXT_PUBLIC_FEATURE_SPONSOR_FORM"false" o "true" para el formulario de patrocinio.

@MariferVL MariferVL requested review from LiliGC and pablolirag May 5, 2025 17:06
LiliGC
LiliGC previously approved these changes May 5, 2025
Copy link
Member

@LiliGC LiliGC left a comment

Choose a reason for hiding this comment

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

He revisado los cambios propuestos en este Pull Request y me parecen adecuados. Los ajustes realizados resuelven los siguientes issues:

Issue #1: La adición del disclaimer para 2025 es clara y necesaria.
Issue #2: La modificación de la data de sponsors para permitir un enlace en Next.js es un buen enfoque, y me parece acertado habilitarlo una vez que se confirmen los sponsors para 2025.
Issue #3: El ajuste en el texto de los botones en la sección /talks mejora la presentación de la información, mostrando primero el nombre de la ciudad y luego la fecha.

Agradezco el trabajo realizado Marifer🙌 y apruebo este Pull Request.

@pablolirag
Copy link
Member

Hay un texto todavia existe en esta version? ""patrocinadores 2024"" ver la imagen,
Uploading Screenshot 2025-05-05 at 14-21-53 PyDay Chile 2025.png…

@MariferVL
Copy link
Contributor Author

@pablolirag por favor revisar #1

LiliGC
LiliGC previously approved these changes May 5, 2025
Copy link
Member

@LiliGC LiliGC left a comment

Choose a reason for hiding this comment

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

Apruebo el pull request se hicieron las modificaciones mencionadas en los issues. Y los patrocinadores se mostrarán cuando estén confirmados.

Copy link
Member

@LiliGC LiliGC left a comment

Choose a reason for hiding this comment

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

Aprobado el PR Marifer. Se aplicaron los cambios comentados en los issues y además se subió el archivo deploy.yml.

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

Probado en local! funcionando

@MariferVL MariferVL merged commit 8384c76 into main May 6, 2025
@MariferVL MariferVL deleted the feat/update-sponsors-talks-buttons branch May 6, 2025 01:10
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.

Ordenar cronológicamente charlas Agregar URL a patrocinantes Año incorrecto en sección patrocinadores

3 participants