From 2390b9d9b103b76f035a9ff860fc548b08b5a77b Mon Sep 17 00:00:00 2001 From: Paul D'Ambra Date: Thu, 11 Jan 2024 10:34:59 +0000 Subject: [PATCH] revert "feat: remember replay filters" (#19707) Revert "feat: remember replay filters" --- .../sessionRecordingsPlaylistLogic.test.ts | 2 -- .../playlist/sessionRecordingsPlaylistLogic.ts | 1 - .../api/test/__snapshots__/test_insight.ambr | 18 ++++++++++++++++++ 3 files changed, 18 insertions(+), 3 deletions(-) diff --git a/frontend/src/scenes/session-recordings/playlist/sessionRecordingsPlaylistLogic.test.ts b/frontend/src/scenes/session-recordings/playlist/sessionRecordingsPlaylistLogic.test.ts index d9b98c4f8309b..0ac5920ecd5bf 100644 --- a/frontend/src/scenes/session-recordings/playlist/sessionRecordingsPlaylistLogic.test.ts +++ b/frontend/src/scenes/session-recordings/playlist/sessionRecordingsPlaylistLogic.test.ts @@ -145,8 +145,6 @@ describe('sessionRecordingsPlaylistLogic', () => { updateSearchParams: true, }) logic.mount() - // the logic persists its filters - logic.actions.resetFilters() }) describe('core assumptions', () => { diff --git a/frontend/src/scenes/session-recordings/playlist/sessionRecordingsPlaylistLogic.ts b/frontend/src/scenes/session-recordings/playlist/sessionRecordingsPlaylistLogic.ts index 90d9562a4bf7d..961178e00f9f0 100644 --- a/frontend/src/scenes/session-recordings/playlist/sessionRecordingsPlaylistLogic.ts +++ b/frontend/src/scenes/session-recordings/playlist/sessionRecordingsPlaylistLogic.ts @@ -365,7 +365,6 @@ export const sessionRecordingsPlaylistLogic = kea ({ ...state, diff --git a/posthog/api/test/__snapshots__/test_insight.ambr b/posthog/api/test/__snapshots__/test_insight.ambr index 802090b19aaba..0dbd61eb66823 100644 --- a/posthog/api/test/__snapshots__/test_insight.ambr +++ b/posthog/api/test/__snapshots__/test_insight.ambr @@ -1541,6 +1541,24 @@ LIMIT 21 /*controller='project_insights-list',route='api/projects/%28%3FP%3Cparent_lookup_team_id%3E%5B%5E/.%5D%2B%29/insights/%3F%24'*/ ''' # --- +# name: TestInsight.test_listing_insights_does_not_nplus1.30 + ''' + SELECT "posthog_taggeditem"."id", + "posthog_taggeditem"."tag_id", + "posthog_taggeditem"."dashboard_id", + "posthog_taggeditem"."insight_id", + "posthog_taggeditem"."event_definition_id", + "posthog_taggeditem"."property_definition_id", + "posthog_taggeditem"."action_id", + "posthog_taggeditem"."feature_flag_id" + FROM "posthog_taggeditem" + WHERE "posthog_taggeditem"."insight_id" IN (1, + 2, + 3, + 4, + 5 /* ... */) /*controller='project_insights-list',route='api/projects/%28%3FP%3Cparent_lookup_team_id%3E%5B%5E/.%5D%2B%29/insights/%3F%24'*/ + ''' +# --- # name: TestInsight.test_listing_insights_does_not_nplus1.4 ''' SELECT "posthog_team"."id",