Skip to content

Rename dashboard widget key active_timerstimers and add migration#679

Merged
multiplex55 merged 1 commit intomasterfrom
codex/update-timers-widget-registration-and-config
Jan 13, 2026
Merged

Rename dashboard widget key active_timerstimers and add migration#679
multiplex55 merged 1 commit intomasterfrom
codex/update-timers-widget-registration-and-config

Conversation

@multiplex55
Copy link
Copy Markdown
Owner

Motivation

  • Standardize the timers widget key to timers and update default dashboard layout to the new name.
  • Ensure existing user configs that still reference the legacy active_timers key continue to load without data loss.

Description

  • Register the timers widget under the new name by changing the entry in WidgetRegistry::with_defaults() from "active_timers" to "timers" in src/dashboard/widgets/mod.rs.
  • Change the default dashboard slot in DashboardConfig::default() to use "timers" in src/dashboard/config.rs.
  • Add migrate_active_timers_widgets() and invoke it from DashboardConfig::sanitize() to detect slot.widget == "active_timers", rewrite it to "timers", merge the default settings via merge_json, and append a migration warning string.

Testing

  • No automated tests were executed as part of this change.
  • The code compiles locally as part of development (manual verification), but cargo test was not run in this rollout.

Codex Task

@multiplex55 multiplex55 merged commit af55d31 into master Jan 13, 2026
1 check passed
@multiplex55 multiplex55 deleted the codex/update-timers-widget-registration-and-config branch January 14, 2026 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant