Skip to content

Fix analytics ui time zone - #59

Merged
Aliemeka merged 1 commit into
mainfrom
fix-analytics-timezone
May 28, 2026
Merged

Aliemeka merged 1 commit into
mainfrom
fix-analytics-timezone

Conversation

@Aliemeka

Copy link
Copy Markdown
Owner

No description provided.

@netlify

netlify Bot commented May 28, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for spit-sh ready!

Name Link
🔨 Latest commit b43f481
🔍 Latest deploy log https://app.netlify.com/projects/spit-sh/deploys/6a18151ad9b2070008fd06ff
😎 Deploy Preview https://deploy-preview-59--spit-sh.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adjusts analytics chart date handling so backend UTC timestamps are parsed correctly in the frontend and adds a custom tooltip label for click analytics.

Changes:

  • Parses naive backend analytics timestamps as UTC instead of local time.
  • Adds formatted tooltip labels with date ordinals and compact hourly labels.
  • Replaces the default area chart tooltip with ChartTooltipContent in the clicks hero chart.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
frontend/lib/analytics/intervals.ts Adds UTC parsing and tooltip label formatting helpers.
frontend/components/analytics/ClicksHeroChart.tsx Uses the new tooltip formatter with the chart tooltip UI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread frontend/lib/analytics/intervals.ts
@Aliemeka
Aliemeka merged commit d7f4517 into main May 28, 2026
6 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.

2 participants