Skip to content

fix(calendar-dropdown): proteger acceso a localStorage en parseEvents#765

Merged
Jonathangarrido merged 1 commit intomainfrom
fix/events-error-sentry
Mar 26, 2026
Merged

fix(calendar-dropdown): proteger acceso a localStorage en parseEvents#765
Jonathangarrido merged 1 commit intomainfrom
fix/events-error-sentry

Conversation

@Jonathangarrido
Copy link
Copy Markdown
Collaborator

Descripción

Se protege el acceso a localStorage en CalendarDropdown dentro de parseEvents para evitar errores en entornos donde el storage no está disponible o su acceso falla. Además, se agregan comentarios para documentar qué persiste cada clave y qué representa cada grupo de eventos.

Motivación y contexto

Sentry estaba reportando un error asociado al uso de localStorage dentro del flujo del dropdown de calendario. El componente dependía de lecturas y escrituras directas al storage, por lo que ante un SecurityError, restricciones del navegador o falta de disponibilidad del objeto, el hook podía romper la ejecución.

Con este ajuste el componente mantiene su comportamiento principal y, si no puede usar localStorage, funciona sin persistencia local:

  • sigue mostrando los eventos
  • sigue separando hoy, mañana y próximos
  • no guarda entre sesiones qué eventos ya fueron vistos
  • el indicador de "nuevo" puede recalcularse nuevamente al no existir persistencia

Tipos de cambios

  • Bugfix
  • Feature
  • Breaking change

@Jonathangarrido Jonathangarrido merged commit 08c5fa2 into main Mar 26, 2026
9 checks passed
@Jonathangarrido Jonathangarrido deleted the fix/events-error-sentry branch March 26, 2026 19:00
github-actions bot pushed a commit that referenced this pull request Mar 26, 2026
## [1.58.3](v1.58.2...v1.58.3) (2026-03-26)

### Bug Fixes

* **calendar-dropdown:** proteger acceso a localStorage en parseEvents ([#765](#765)) ([08c5fa2](08c5fa2))
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.

1 participant