Skip to content

Incluir convence-a-tu-jefe en sitemap y validador - #148

Open
yosef7 wants to merge 1 commit into
mainfrom
fix/127-sitemap-convence
Open

yosef7 wants to merge 1 commit into
mainfrom
fix/127-sitemap-convence

Conversation

@yosef7

@yosef7 yosef7 commented Sep 27, 2026 •

Copy link
Copy Markdown
Member

Añade https://pycon.pa/2026/convence-a-tu-jefe.html a sitemap.xml y a SITEMAP_REQUIRED. Usa como lastmod el 18 de septiembre, fecha del último cambio de la página en main, y comprueba que ambas listas coincidan sin duplicados.

Closes #127

Para cumplir la validación completa, incorpora la corrección puntual del parser ya propuesta en #124: preconnect y dns-prefetch indican orígenes, no recursos HTTP. La comprobación de hojas CSS, enlaces normales y demás recursos continúa activa. Esto elimina los falsos 404 de las raíces de Google Fonts sin excluir esos dominios de la validación.

Validación:

  • python3 scripts/validate_site.py: correcto, incluidas las comprobaciones externas de las 12 páginas.
  • Comprobación dirigida del parser: omite sugerencias de conexión y conserva CSS y enlaces normales, incluso hacia los mismos hosts.
  • Sitemap y SITEMAP_REQUIRED coinciden, sin duplicados.
  • git diff --cached --check: correcto.

Solo cambian sitemap.xml y scripts/validate_site.py; no se modifica la página ni se restituye la agenda.


Revisión conjunta de las PR pendientes: PR borrador #153 (NO FUSIONAR) muestra este cambio junto con los demás y documenta los conflictos resueltos y las pruebas. Esta PR individual conserva su aprobación y fusión propias.

@yosef7
yosef7 requested a review from a team September 27, 2026 14:54
@netlify

netlify Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for pycon ready!

Name Link
🔨 Latest commit 974ceb4
🔍 Latest deploy log https://app.netlify.com/projects/pycon/deploys/6ab92e2d1a0420000808c4e0
😎 Deploy Preview https://deploy-preview-148--pycon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

Incluir convence-a-tu-jefe.html en sitemap.xml y en el validador

2 participants