WS8 PR D: review round 1 - UTC dates, future-event bound, comments
Independent review (approve-with-fixes, all Minor): 1. Dashboard formatUkDate now renders lastActive/lastRun in UTC (timeZone: "UTC") so dates can't drift a day from the UTC-bucketed trend for late-night UK viewers. 2. usageStats: added an upper bound - a future-dated created_at (clock skew / bad data) is treated as no timestamp everywhere (local eventTime helper), so it can't count in tiles while falling outside the daily series. New test. 3. usageStats: commented the "Untitled workflow" fallback as defensive-only (titleById is always populated by the has() guard; workflows.title is NOT NULL) - kept for safety. 4. BUILD_LOG: recorded the definition-of-done deviation that the chart/tables were not screenshotted in a running app (geometry validated by reasoning + mock-up parity). Backend tsc clean; vitest 235 passed (20 files). Frontend tsc clean. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
| Repository | Sloth-ninja/JessicaOSS |
|---|---|
| Author | sloth-ninja <ezanahaddis@gmail.com> |
| Authored | |
| Parents | 863f8447 |
| Stats | 4 files changed , +40 , -5 |
| Part of | WS8 PR D: admin usage dashboard |
Capture this commit into my fork
Download a Markdown prompt that tells Claude how to port this
exact commit into your working tree. Run it via
claude -p < capture-commit-22699fe5.md
from inside the repo you want the change in.