Skip to content

Hotfix/correccion zona horaria - #10

Merged
00Ssam00 merged 2 commits into
mainfrom
hotfix/correccion-zona-horaria
Oct 16, 2025
Merged

Hotfix/correccion zona horaria#10
00Ssam00 merged 2 commits into
mainfrom
hotfix/correccion-zona-horaria

Conversation

@00Ssam00

Copy link
Copy Markdown
Owner

Descripción

Este PR corrige definitivamente el problema de desfase de zona horaria en el heatmap. El fix anterior ( PR #8 ) intentó solucionar la generación de nuevo día mediante conversiones manuales entre UTC y Colombia (UTC-5), pero esto generaba inconsistencias y desfases en la asignación de commits a días específicos.

Cambios principales:

  • Eliminado completamente el método adjustToBogotaDate() que causaba conversiones incorrectas
  • Removidas todas las conversiones manuales entre UTC y zona horaria local
  • Simplificado el cálculo de rangos de fechas usando únicamente la fecha/hora del sistema local
  • Unificados getLastWeekCommits() y getLastMonthCommits() para usar el método común extractDays()
  • Los commits ahora se asignan correctamente según las fechas en formato YYYY-MM-DD que devuelve la API de GitHub
  • Uso directo de Date nativo sin manipulaciones de offset manual

¿Por qué este enfoque es correcto?

La API de GitHub devuelve las fechas de contribución ya en formato YYYY-MM-DD sin información de zona horaria. Al trabajar directamente con Date del sistema y convertir a ISO, obtenemos la fecha correcta sin necesidad de conversiones manuales que introducían errores de desfase.

Soluciona completamente el problema nº #5
Closes #5

Tipo de cambio

  • Corrección de error (cambio no disruptivo que soluciona un problema)
  • Nueva funcionalidad (cambio no disruptivo que agrega una característica)
  • Cambio importante (corrección o característica que puede causar que la funcionalidad existente deje de funcionar como se espera)
  • Actualización de documentación

¿Cómo se ha probado esto?

Las pruebas se realizaron verificando los logs en consola y validando el comportamiento del heatmap en tiempo real.

  • Prueba A: Verificado que las fechas se calculan correctamente para los últimos 7 días
  • Prueba B: Verificado que las fechas se calculan correctamente para los últimos 30 días
  • Prueba C: Verificado que los commits se asignan al día correcto sin desfase
  • Prueba D: Verificado comportamiento en diferentes horas del día (medianoche, mediodía, etc.)

Lista de verificación

  • Mi código sigue las guías de estilo de este proyecto
  • He realizado una autoevaluación de mi propio código
  • He comentado mi código, especialmente en las partes difíciles de entender
  • He realizado los cambios correspondientes en la documentación
  • Mis cambios no generan nuevas advertencias
  • He añadido pruebas que demuestran que mi corrección funciona correctamente (validación mediante logs y pruebas manuales)
  • Las pruebas unitarias nuevas y existentes pasan correctamente en mi entorno local
  • Cualquier cambio dependiente ha sido fusionado y publicado en los módulos relacionados

Notas adicionales:

  • Se eliminaron ~100 líneas de código relacionadas con conversiones manuales de zona horaria
  • El código es ahora más simple, legible y mantenible
  • Se mantiene compatibilidad total con ambas vistas: Week y Month
  • Los logs son más claros y concisos
  • El método extractDays() es reutilizable para cualquier período de días

Archivos modificados:

  • src/services/github-service.ts: Refactorización completa del manejo de fechas

00Ssam00 and others added 2 commits October 15, 2025 21:11
Se elimino el codigo anterior, usado para pruebas de zona horaria
@00Ssam00 00Ssam00 self-assigned this Oct 16, 2025
@00Ssam00
00Ssam00 merged commit 9264681 into main Oct 16, 2025
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.

Problema con la generación de nuevo día en el heatmap según hora colombiana (UTC-5)

1 participant