Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Custom day until now in Web Analytics #28360

Merged

Conversation

cosmeoes
Copy link
Contributor

@cosmeoes cosmeoes commented Feb 6, 2025

Problem

Closes #28171

Changes

It now allows you to choose the time and applies the filter.
image

image

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Does this work well for both Cloud and self-hosted?

How did you test this code?

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR adds time precision support to the Web Analytics dashboard's date filter, allowing users to select specific hours and minutes when using the 'Custom date until now' option.

  • Added allowTimePrecision={true} prop to DateFilter component in /frontend/src/scenes/web-analytics/WebAnalyticsDashboard.tsx to enable hour/minute selection
  • Fixes issue where custom date-to-now selections were defaulting to 12 AM regardless of selected time
  • Leverages existing time precision functionality from the DateFilter component to properly apply selected times

💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@cosmeoes cosmeoes force-pushed the cosmeoes/fix-custom-date-until-now branch from 203c975 to 8724163 Compare February 6, 2025 05:31
@Twixes Twixes requested a review from a team February 6, 2025 11:30
Copy link
Member

@robbie-c robbie-c left a comment

Choose a reason for hiding this comment

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

Thanks!

@robbie-c robbie-c merged commit 3632661 into PostHog:master Feb 6, 2025
100 of 102 checks passed
@cosmeoes cosmeoes deleted the cosmeoes/fix-custom-date-until-now branch February 7, 2025 02:35
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.

Custom date until now... hour/minute options don't apply
2 participants