Skip to content

Commit

Permalink
chore: wrap setFilters to avoid crash (PostHog#29045)
Browse files Browse the repository at this point in the history
  • Loading branch information
veryayskiy authored Feb 21, 2025
1 parent c48b676 commit 6eed2f6
Showing 1 changed file with 15 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import { featureFlagLogic } from 'lib/logic/featureFlagLogic'
import { objectClean, objectsEqual } from 'lib/utils'
import { eventUsageLogic } from 'lib/utils/eventUsageLogic'
import { getCurrentTeamId } from 'lib/utils/getAppContext'
import posthog from 'posthog-js'

import { activationLogic, ActivationTask } from '~/layout/navigation-3000/sidepanel/panels/activation/activationLogic'
import { NodeKind, RecordingOrder, RecordingsQuery, RecordingsQueryResponse } from '~/queries/schema/schema-general'
Expand Down Expand Up @@ -468,13 +469,20 @@ export const sessionRecordingsPlaylistLogic = kea<sessionRecordingsPlaylistLogic
{ persist: true, prefix: `${getCurrentTeamId()}__${key}` },
{
setFilters: (state, { filters }) => {
if (!isValidRecordingFilters(filters)) {
console.error('Invalid filters provided:', filters)
return state
}
return {
...state,
...filters,
try {
if (!isValidRecordingFilters(filters)) {
posthog.captureException(new Error('Invalid filters provided'), {
filters,
})
return getDefaultFilters(props.personUUID)
}
return {
...state,
...filters,
}
} catch (e) {
posthog.captureException(e)
return getDefaultFilters(props.personUUID)
}
},
resetFilters: () => getDefaultFilters(props.personUUID),
Expand Down

0 comments on commit 6eed2f6

Please sign in to comment.