Skip to content

fix: corregir deteccion de cambio de dia en zona horaria Colombia - #8

Merged
00Ssam00 merged 1 commit into
mainfrom
fix/generacion-nuevo-dia
Oct 13, 2025
Merged

fix: corregir deteccion de cambio de dia en zona horaria Colombia#8
00Ssam00 merged 1 commit into
mainfrom
fix/generacion-nuevo-dia

Conversation

@00Ssam00

Copy link
Copy Markdown
Owner

Descripción

Este PR corrige el problema crítico donde el heatmap no generaba un nuevo cuadrado al pasar las 12:00 AM en Colombia (UTC-5). El método adjustToBogotaDate() en github-service.ts estaba aplicando incorrectamente una conversión de zona horaria a fechas que ya estaban en formato local, causando un desfase de un día en la visualización de commits.

Cambios principales:

  • Implementada detección de formato de entrada (fecha simple YYYY-MM-DD vs timestamp ISO 8601 completo)
  • Las fechas simples ya no sufren conversión incorrecta
  • Los timestamps UTC se siguen convirtiendo correctamente a Colombia (UTC-5)
  • Agregado logging diferenciado para facilitar debugging

Soluciona 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 el debugger de Stream Deck y validando el comportamiento del heatmap en tiempo real.

  • Prueba A: Verificado que fechas en formato YYYY-MM-DD no sufren desfase
  • Prueba B: Verificado que timestamps completos se convierten correctamente
  • Prueba C: Verificado que commits se asignan al día correcto en Colombia
  • Prueba D: Verificado que el rango solicitado usa correctamente medianoche Colombia (00:00)

Instrucciones para reproducir:

  1. Compilar el plugin: npm run build
  2. Reiniciar Stream Deck
  3. Presionar el botón del heatmap
  4. Abrir debugger
  5. Verificar logs de conversión en la consola

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 o funcionalidad funciona correctamente (validación mediante logs)
  • 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:

  • El fix usa regex para detectar formato de entrada: /^\d{4}-\d{2}-\d{2}$/
  • Se mantiene compatibilidad con ambas vistas: Week y Month
  • Los logs de debugging facilitan validación futura del comportamiento

- Detectar formato de entrada (fecha simple vs timestamp completo)
- Evitar conversion incorrecta de fechas ya en formato local
- Mantener conversion UTC a Colombia para timestamps
- Agregar logging diferenciado para debugging
- Garantizar generacion de nuevo cuadrado a las 00:00 Colombia

Los commits ahora se asignan correctamente al dia local y el heatmap
genera un nuevo cuadrado al cruzar la medianoche en Colombia (UTC-5).

Resuelve #5
@00Ssam00 00Ssam00 self-assigned this Oct 13, 2025
@00Ssam00
00Ssam00 merged commit fde71a8 into main Oct 13, 2025
@00Ssam00 00Ssam00 changed the title fix: corregir deteccion de cambio de dia en zona horaria Colombia Merge fix: corregir deteccion de cambio de dia en zona horaria Colombia Oct 13, 2025
@00Ssam00 00Ssam00 changed the title Merge fix: corregir deteccion de cambio de dia en zona horaria Colombia fix: corregir deteccion de cambio de dia en zona horaria Colombia Oct 13, 2025
@00Ssam00 00Ssam00 mentioned this pull request Oct 16, 2025
12 tasks
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