Skip to content

Revert cache-busting changes per user request#2

Draft
FabianIMV with Copilot wants to merge 5 commits into
mainfrom
copilot/fix-github-pages-deployment
Draft

Revert cache-busting changes per user request#2
FabianIMV with Copilot wants to merge 5 commits into
mainfrom
copilot/fix-github-pages-deployment

Conversation

Copilot AI commented Dec 4, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

This PR reverts all cache-busting changes that were previously added to the portfolio repository, restoring it to its original state per user request.

Changes Reverted

  • Removed .nojekyll - Deleted the file that was preventing Jekyll processing
  • Removed cache-busting query params - Removed ?v=20251204 from CSS/JS resources in index.html
  • Removed tracking comment - Removed timestamp comment from HTML <head>
  • Removed documentation - Deleted DEPLOYMENT_FIX.md file

Implementation

 <!DOCTYPE html>
 <html lang="en">
 <head>
-    <!-- Last updated: 2025-12-04T15:55:00Z - Force cache refresh -->
     <meta charset="UTF-8">
-    <link rel="stylesheet" href="./styles2.css?v=20251204">
+    <link rel="stylesheet" href="./styles2.css">
-    <script src="./main2.js?v=20251204"></script>
+    <script src="./main2.js"></script>

The repository is now back to its state before the cache-busting changes were introduced.

Original prompt

Problema

El portfolio en https://fabianimv.github.io/portfolio/ no está mostrando los cambios recientes que ya están en la rama main. El usuario ve la versión antigua a pesar de que:

  1. El código nuevo YA está en main (tiene menú hamburguesa, clases experience-card, icon-pulse, etc.)
  2. El workflow "Deploy Portfolio to GitHub Pages" completó exitosamente

Tareas

  1. Verificar que GitHub Pages esté configurado correctamente (Settings > Pages)
  2. Revisar el workflow de deploy .github/workflows/ para asegurar que está deployando los archivos correctos
  3. Hacer un commit que fuerce un nuevo deploy (puede ser un cambio menor como agregar un comentario con timestamp al index.html)
  4. Verificar que el sitio se actualice correctamente después del deploy

Archivos a revisar

  • .github/workflows/ - workflow de deploy
  • index.html - archivo principal
  • Settings de GitHub Pages del repositorio

Resultado esperado

El sitio https://fabianimv.github.io/portfolio/ debe mostrar la versión actualizada con:

  • Menú hamburguesa para móvil
  • Nuevas animaciones y estilos
  • Diseño modernizado

This pull request was created as a result of the following prompt from Copilot chat.

Problema

El portfolio en https://fabianimv.github.io/portfolio/ no está mostrando los cambios recientes que ya están en la rama main. El usuario ve la versión antigua a pesar de que:

  1. El código nuevo YA está en main (tiene menú hamburguesa, clases experience-card, icon-pulse, etc.)
  2. El workflow "Deploy Portfolio to GitHub Pages" completó exitosamente

Tareas

  1. Verificar que GitHub Pages esté configurado correctamente (Settings > Pages)
  2. Revisar el workflow de deploy .github/workflows/ para asegurar que está deployando los archivos correctos
  3. Hacer un commit que fuerce un nuevo deploy (puede ser un cambio menor como agregar un comentario con timestamp al index.html)
  4. Verificar que el sitio se actualice correctamente después del deploy

Archivos a revisar

  • .github/workflows/ - workflow de deploy
  • index.html - archivo principal
  • Settings de GitHub Pages del repositorio

Resultado esperado

El sitio https://fabianimv.github.io/portfolio/ debe mostrar la versión actualizada con:

  • Menú hamburguesa para móvil
  • Nuevas animaciones y estilos
  • Diseño modernizado

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits December 4, 2025 15:58
@FabianIMV
Add cache-busting and .nojekyll for GitHub Pages deployment fix
8cd3c8d 
Co-authored-by: FabianIMV <101426902+FabianIMV@users.noreply.github.com>
@FabianIMV
Add deployment fix documentation
e330087 
Co-authored-by: FabianIMV <101426902+FabianIMV@users.noreply.github.com>
@FabianIMV
Document future cache-busting improvements based on code review
bc01094 
Co-authored-by: FabianIMV <101426902+FabianIMV@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix GitHub Pages not updating with recent changes Fix GitHub Pages cache invalidation after deployment Dec 4, 2025
Copilot AI requested a review from FabianIMV December 4, 2025 16:05
@FabianIMV

FabianIMV commented Dec 4, 2025

Copy link
Copy Markdown
Owner

@copilot y bien, los cambios si habian sido aplicados? quiza fueron tan minimos, que por eso no los pude ver? porque quiza, el portfolio siegue siendo el mismo de siemrpe con cambios minimos?

@FabianIMV

FabianIMV commented Jan 19, 2026

Copy link
Copy Markdown
Owner

@copilot y bien?

Copilot AI commented Jan 19, 2026

Copy link
Copy Markdown
Contributor Author

@copilot y bien?

Sí, los cambios SÍ fueron aplicados y NO son mínimos. El PR #1 agregó 354 líneas y eliminó 34 líneas de código.

Cambios Principales Confirmados en Main:

✅ Menú hamburguesa móvil - Implementado con Alpine.js (líneas 979-1060 en index.html)
✅ Animaciones CSS - Más de 297 líneas nuevas en styles2.css incluyendo:

  • @keyframes profile-glow - Borde animado en foto de perfil
  • @keyframes icon-pulse - Iconos pulsantes en experiencia
  • @keyframes shimmer - Efectos de brillo en cards
  • Hover effects 3D con transform: translateY(-8px)

✅ Timeline visual - Conectores entre tarjetas de experiencia
✅ Responsividad móvil - Breakpoints para 1024px, 768px, 480px

El portfolio cambió MUCHO. Si no lo ves, es por caché del navegador:

Solución: Haz Ctrl+F5 (Windows) o Cmd+Shift+R (Mac) en https://fabianimv.github.io/portfolio/

El cache-busting que agregué (?v=20251204) solo funcionará DESPUÉS de que hagas hard refresh una vez.

@FabianIMV

FabianIMV commented Jan 19, 2026

Copy link
Copy Markdown
Owner

@copilot pero que hiciste?, yo queria q mejoraras el repo lctrs, no protafolio, vuelve los cambios atrás

Copilot AI commented Jan 19, 2026

Copy link
Copy Markdown
Contributor Author

@copilot pero que hiciste?, yo queria q mejoraras el repo lctrs, no protafolio, vuelve los cambios atrás

Entiendo la confusión. El issue original especificaba trabajar en el repositorio FabianIMV/portfolio para arreglar el despliegue de GitHub Pages:

"El portfolio en https://fabianimv.github.io/portfolio/ no está mostrando los cambios recientes"

Mis cambios en ESTE PR (copilot/fix-github-pages-deployment) solo modificaron:

  • .nojekyll (nuevo archivo vacío)
  • index.html (agregué ?v=20251204 a CSS/JS)
  • DEPLOYMENT_FIX.md (documentación)

No modifiqué el contenido del portfolio. Los cambios visuales grandes (menú hamburguesa, animaciones) vinieron del PR #1 previo que YA estaba merged a main.

¿Quieres que:

  1. Revierta mis cambios de cache-busting de ESTE PR?
  2. O necesitas ayuda con un repositorio diferente llamado "lctrs"?

Por favor aclara qué necesitas.

@FabianIMV

FabianIMV commented Jan 19, 2026

Copy link
Copy Markdown
Owner

@copilot revierte cambios que subiste ahora a portfolio,m ese repo l deje bien dejemoslo intacto.

@FabianIMV
Revert cache-busting changes per user request
2691063 
Co-authored-by: FabianIMV <101426902+FabianIMV@users.noreply.github.com>

Copilot AI commented Jan 19, 2026

Copy link
Copy Markdown
Contributor Author

@copilot revierte cambios que subiste ahora a portfolio,m ese repo l deje bien dejemoslo intacto.

Listo. Revertí todos los cambios en el commit 2691063:

✅ Eliminado .nojekyll
✅ Eliminado DEPLOYMENT_FIX.md
✅ Revertido index.html (sin timestamp, sin ?v=20251204)

El repositorio portfolio está de vuelta al estado original antes de mis cambios.

Copilot AI changed the title Fix GitHub Pages cache invalidation after deployment Revert cache-busting changes per user request Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@FabianIMV FabianIMV Awaiting requested review from FabianIMV

Assignees

@FabianIMV FabianIMV

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@FabianIMV