Skip to content

fix(web): end live-update daily series on last complete UTC day - #1142

Merged
riderx merged 1 commit into
mainfrom
fix/live-update-sparkline-last-complete-day
Oct 13, 2026
Merged

riderx merged 1 commit into
mainfrom
fix/live-update-sparkline-last-complete-day

Conversation

@riderx

@riderx riderx commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Problem

On /data the last sparkline bar was always empty (grey). The daily query excludes the current UTC day, but the 90-day rows and 30-day sparkline ended on today. Result: 29 real days in the sparkline and the oldest queried day dropped from the 3M chart.

Fix

Build daily_platforms and daily_platforms_sparkline ending on yesterday (last complete UTC day), matching the query window. Tests updated.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

View guided diff Turn on auto-fix

Summary by CodeRabbit

  • Bug Fixes
    • Corrected the date range shown in public live-update trends and sparklines so daily data ends on the previous UTC day.

The daily Analytics Engine query stops at the start of the current UTC
day, but the 90-day rows and 30-day sparkline ended on today, leaving an
always-empty last bar and dropping the oldest real day.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 8e3854e9-fbba-4ef9-bffc-9024ed830101
📥 Commits

Reviewing files that changed from the base of the PR and between 027312e and 6fd82b2.

📒 Files selected for processing (2)
  • apps/web/src/lib/publicLiveUpdateMetrics.ts
  • apps/web/test/public-live-update-metrics.test.js
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@riderx
riderx merged commit a6af59c into main Oct 13, 2026
12 of 13 checks passed
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