Skip to content

Experimental engaged session switch - #6610

Open
ukutaht wants to merge 2 commits into
masterfrom
engaged-session-switch
Open

Experimental engaged session switch#6610
ukutaht wants to merge 2 commits into
masterfrom
engaged-session-switch

Conversation

@ukutaht

@ukutaht ukutaht commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Background

An experiment to see how it would look like to add a 'engaged sessions only' filter to deal with bot traffic. An engaged session is defined as a session with at least one engagement event.

This is intentionally the smallest way to test this in production. It is meant to be easily revertable. If the experiment is a success in that the filter:

  • Helps clean up dashboards with spam
  • Does not exclude too much real human traffic

Then the plan is to revert this commit and think about how to build this properly. Currently it ignores many considerations including realtime data, imported data, and historical data before engagement events were even added.

Changes

For superadmins only, adds a UI switch that always applies a [:has_done, [:is, "event:name", ["engagement"]] filter to all API requests. Does not apply to the current visitors number and when enabled, imported data is inaccessible due to backend rules.

Screenshot 2026-08-19 at 23 12 18

Tests

  • Automated tests have been added

Changelog

  • This PR does not make a user-facing change (experiment for super-admins only)

Documentation

  • This change does not need a documentation update

Dark mode

  • The UI has been tested both in dark and light mode

@ukutaht
ukutaht force-pushed the engaged-session-switch branch from b973bdc to 6aac6a9 Compare August 19, 2026 20:20
@ukutaht
ukutaht requested a review from a team August 19, 2026 20:20
@github-actions

Copy link
Copy Markdown
Preview environment👷🏼‍♀️🏗️
PR-6610

@aerosol aerosol left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Interesting idea, let's see how it behaves

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants