PostHog / posthog

🦔 PostHog provides open-source web & product analytics, session recording, feature flagging and A/B testing that you can self-host. Get started - free.
https://posthog.com
Other
22.3k stars 1.35k forks source link

fix(alerts): pass correct filters for aggregate trends #26357

Closed anirudhpillai closed 6 days ago

anirudhpillai commented 6 days ago

Problem

After recent changes, aggregate trends alerts (pie chart...) were only checking for last week/month instead of full date range (eg. last 8 weeks) while evaluating alerts

Changes

Reset to checking full date range, this ensures the insight value user sees in the alert email/alert modal matches that when they open the insight.

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

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

N/A

How did you test this code?

Added tests