Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: allow triggering sessions when events occur #1523

Merged
merged 12 commits into from
Nov 15, 2024
Merged

Conversation

richard-better
Copy link
Contributor

@richard-better richard-better commented Nov 13, 2024

folk frequently want to control their cost and volume of recordings

this allows them to set a config to start replay "paused" and only start sending data to the server when specified events are seen

Client side of PostHog/posthog#26168

Copy link

vercel bot commented Nov 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
posthog-js ✅ Ready (Inspect) Visit Preview Nov 14, 2024 4:50pm

Copy link

github-actions bot commented Nov 13, 2024

Size Change: +8.19 kB (+0.26%)

Total Size: 3.11 MB

Filename Size Change
dist/array.full.es5.js 254 kB +910 B (+0.36%)
dist/array.full.js 356 kB +808 B (+0.23%)
dist/array.full.no-external.js 355 kB +808 B (+0.23%)
dist/array.js 171 kB +809 B (+0.48%)
dist/array.no-external.js 170 kB +809 B (+0.48%)
dist/main.js 172 kB +809 B (+0.47%)
dist/module.full.js 355 kB +808 B (+0.23%)
dist/module.full.no-external.js 355 kB +808 B (+0.23%)
dist/module.js 171 kB +809 B (+0.48%)
dist/module.no-external.js 170 kB +809 B (+0.48%)
ℹ️ View Unchanged
Filename Size
dist/all-external-dependencies.js 204 kB
dist/dead-clicks-autocapture.js 13 kB
dist/exception-autocapture.js 8.8 kB
dist/external-scripts-loader.js 2.19 kB
dist/recorder-v2.js 113 kB
dist/recorder.js 113 kB
dist/surveys-preview.js 56.7 kB
dist/surveys.js 62.1 kB
dist/tracing-headers.js 1.33 kB
dist/web-vitals.js 10.3 kB

compressed-size-action

Copy link
Member

@pauldambra pauldambra left a comment

Choose a reason for hiding this comment

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

some thoughts but otherwise...

👍

@pauldambra pauldambra marked this pull request as ready for review November 13, 2024 20:02
@pauldambra pauldambra added the bump minor Bump minor version when this PR gets merged label Nov 13, 2024
@pauldambra pauldambra merged commit 8db1006 into main Nov 15, 2024
13 checks passed
@pauldambra pauldambra deleted the event-triggers branch November 15, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump minor Bump minor version when this PR gets merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants