Skip to content

fix(in-app analytics) ignore unavailable filters - #1202

Merged
siiick merged 3 commits into
mainfrom
fix/in-app-analytics-ignore-unavailable-filters
Nov 7, 2025
Merged

fix(in-app analytics) ignore unavailable filters#1202
siiick merged 3 commits into
mainfrom
fix/in-app-analytics-ignore-unavailable-filters

Conversation

@siiick

@siiick siiick commented Nov 6, 2025

Copy link
Copy Markdown
Contributor
  • Filters out unavailable filters from backend requests params.
  • UI : Grey out unavailable filters even if they have value set.

@sentry

sentry Bot commented Nov 6, 2025

Copy link
Copy Markdown

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: packages/app-builder/src/routes/_builder+/_analytics+/analytics.$scenarioId.tsx

Function Unhandled Issue
Analytics TypeError: (x ?? []).map is not a function routes...
Event Count: 1 Affected Users: 1

@Pascal-Delange Pascal-Delange left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@siiick
siiick force-pushed the fix/in-app-analytics-ignore-unavailable-filters branch from 62ee5d6 to 817aea8 Compare November 7, 2025 08:02
@siiick
siiick enabled auto-merge (squash) November 7, 2025 08:03
@siiick
siiick merged commit 65d96dc into main Nov 7, 2025
5 checks passed
@siiick
siiick deleted the fix/in-app-analytics-ignore-unavailable-filters branch November 7, 2025 08:04
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.

2 participants