Skip to content

Conversation

@alephramos
Copy link
Contributor

📝 Descripción

Breve descripción de los cambios realizados en este PR.

🎯 Tipo de Cambio

  • 🐛 Bug fix (cambio que arregla un problema)
  • ✨ Nueva característica (cambio que agrega funcionalidad)
  • 📚 Mejora de documentación (cambio que mejora la documentación)
  • 🎨 Mejora de diseño (cambio que mejora la apariencia)
  • ⚡ Mejora de rendimiento (cambio que mejora el rendimiento)
  • 🔧 Refactorización (cambio que no arregla un bug ni agrega una característica)
  • 🧪 Pruebas (cambio que agrega o mejora pruebas)
  • 📅 Nuevo meetup (agregar información de un nuevo meetup)

🔗 Issues Relacionados

Closes #[número del issue]
Relacionado con #[número del issue]

📋 Cambios Realizados

  • Cambio 1
  • Cambio 2
  • Cambio 3

🧪 Pruebas Realizadas

  • Pruebas locales ejecutadas
  • Sitio construido correctamente (mkdocs build)
  • Servidor de desarrollo funciona (mkdocs serve)
  • Documentación actualizada (si aplica)
  • Pruebas en diferentes navegadores (si aplica)

📸 Capturas de Pantalla

Si este PR incluye cambios visuales, incluye capturas de pantalla:

Antes

Antes

Después

Después

🔍 Checklist

  • Mi código sigue las guías de estilo del proyecto
  • He realizado una auto-revisión de mi código
  • He comentado mi código, especialmente en áreas difíciles de entender
  • He hecho los cambios correspondientes en la documentación
  • Mis cambios no generan nuevos warnings
  • He agregado pruebas que prueban que mi corrección funciona o que mi característica funciona
  • Las pruebas nuevas y existentes pasan localmente con mis cambios
  • Cualquier cambio dependiente ha sido fusionado y publicado en módulos downstream

📝 Notas Adicionales

Agrega cualquier contexto adicional sobre el PR aquí.

🤝 ¿Necesitas ayuda?

Si tienes preguntas sobre este PR o necesitas ayuda, no dudes en preguntar en los comentarios.


Gracias por contribuir a Python CDMX Charlas! 🐍

erickisos and others added 7 commits October 20, 2025 12:42
* fix: broken links y check script

* chore: run pre-commit
* fix(config): agregar use_directory_urls para manejar URLs sin extensión

- Configurar use_directory_urls: true en mkdocs.yml
- Permite que URLs como /meetups/ funcionen correctamente
- Mantiene compatibilidad con estructura de directorios de MkDocs

* feat(infra): implementar CloudFront Function para URLs sin extensión

- Agregar CloudFront Function url_rewrite para manejar URLs sin .html
- Función redirige /meetups/ a /meetups/index.html automáticamente
- Asociar función con todos los cache behaviors para consistencia
- Soluciona problema de enlaces que no funcionan en producción

* docs: agregar documentación para fix de URLs sin extensión

- Documentar problema de URLs que no funcionan en producción
- Explicar solución implementada con CloudFront Function
- Incluir pasos de despliegue y verificación
- Agregar notas técnicas sobre funcionamiento de la solución
…aging (#45)

* fix(config): agregar use_directory_urls para manejar URLs sin extensión

- Configurar use_directory_urls: true en mkdocs.yml
- Permite que URLs como /meetups/ funcionen correctamente
- Mantiene compatibilidad con estructura de directorios de MkDocs

* feat(infra): implementar CloudFront Function para URLs sin extensión

- Agregar CloudFront Function url_rewrite para manejar URLs sin .html
- Función redirige /meetups/ a /meetups/index.html automáticamente
- Asociar función con todos los cache behaviors para consistencia
- Soluciona problema de enlaces que no funcionan en producción

* docs: agregar documentación para fix de URLs sin extensión

- Documentar problema de URLs que no funcionan en producción
- Explicar solución implementada con CloudFront Function
- Incluir pasos de despliegue y verificación
- Agregar notas técnicas sobre funcionamiento de la solución

* feat(infra): aplicar CloudFront Function a staging environment

- Agregar función url_rewrite a todos los cache behaviors de staging
- Reutilizar la misma CloudFront Function global para consistencia
- Soluciona problema de URLs sin extensión en staging.pythoncdmx.org
- Mantiene configuración de cache más agresiva para staging

* docs: actualizar documentación con soporte para staging

- Agregar información sobre configuración de staging
- Incluir URLs de verificación para ambos ambientes
- Documentar que la misma CloudFront Function se reutiliza
Add CloudFront Functions to rewrite directory URLs to include index.html.
This fixes 404 errors on pages like /meetups/, /comunidad/ponentes/, etc.

Changes:
- Add CloudFront Function for production and staging environments
- Associate functions with default cache behavior on distributions
- Add comprehensive documentation in FIX_404_ERRORS.md

The solution uses CloudFront Functions to intercept viewer requests and
automatically append index.html to URLs ending with / or without file
extensions, enabling MkDocs directory URLs to work correctly with S3/CloudFront.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Merge changes from staging branch and fix the URL rewriting logic
to correctly handle MkDocs directory URLs.

Changes:
- Merge staging branch (commits a111f61, e9b294a)
- Remove duplicate cloudfront-function.tf (use unified url_rewrite function)
- Fix url_rewrite function: change '.html' to '/index.html' for URLs without extension
- Update documentation to reflect correct behavior
- Integrate mkdocs.yml changes from staging

The corrected logic now properly handles MkDocs use_directory_urls:
- /meetups  → /meetups/index.html (was incorrectly /meetups.html)
- /meetups/ → /meetups/index.html (already correct)

This ensures all directory URLs work correctly in both production and staging.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@alephramos alephramos merged commit 5d6c58d into main Oct 26, 2025
2 checks passed
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.

4 participants