PostHog / posthog

🦔 PostHog provides open-source product analytics, session recording, feature flagging and A/B testing that you can self-host.
https://posthog.com
Other
21.2k stars 1.26k forks source link

fix: strip uuid in schema_helpers with the other fe only fields #23376

Closed aspicer closed 3 months ago

aspicer commented 3 months ago

Problem

Fixes https://github.com/PostHog/company-internal/issues/1423

The call to insights to load a saved retention query was often cache missing. This caused the front-end to not get data and launch a query.

Changes

Add uuid to filters_to_query. This allows us to cache hit.

There is probably another bug on the frontend where /query responses aren't getting saved locally on the fe the same way the insights responses are.

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

yes

How did you test this code?

Tested it locally.