chore(analytics): retire routes that only fed the dropped analytics charts #131
Labels
No labels
blocked
bug
enhancement
high-priority
low-priority
needs-info
needs-triage
ready-for-agent
ready-for-human
referenced
research
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
gabogg/hikcentral#131
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Blocked by: the statistics deck replacing the admin
analyticstab (#80 deck UI work).Decided with the maintainer on 2026-09-25 (recorded on #80): the three operational charts on the admin
analyticstab are dropped, not moved, because the statistics deck covers their content:Once the deck ships and those charts are removed, retire any route that only fed them.
Candidates (check callers first)
GET /api/analytics/timeseries/multiday: only caller today is the multi-day comparison inapp.js.GET /api/analytics/timeseries/hourly: fed the flow and occupancy charts. The statistics hourly route may share its service function, so retire the route, not the shared service code.GET /api/analytics/dwell/dayparts: check whether anything besides the old analytics tab uses it; the statistics dayparts route is the deck's.For each, before removing: search the frontend, CLI, scripts and docs for callers; remove the route, its schema-only types, its tests and its entry in
docs/api/README.md; keep shared service code that the statistics routes use.scripts/check_docs.pymust pass with the new operation count.Refs #80.
🤖 Generated with Claude Code
Maintainer requested removal of the blocked label. The statistics deck is present, but current app/static/js/app.js still calls /api/analytics/timeseries/hourly and /api/analytics/timeseries/multiday. Remove or replace the old chart callers before deleting their routes, and retain shared statistics service code. The existing caller-check acceptance criteria remain in force; removing this label does not establish that the old charts have already been removed.