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: Add customization to print events drop them #1572

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

robbie-c
Copy link
Member

@robbie-c robbie-c commented Nov 29, 2024

Changes

Add customization to print all events then drop them.
Fixed #1320

Checklist

  • Tests for new code (see advice on the tests we use)
  • Accounted for the impact of any changes across different browsers
  • Accounted for backwards compatibility of any changes (no breaking changes in posthog-js!)

Copy link

vercel bot commented Nov 29, 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 29, 2024 4:35pm

@robbie-c robbie-c changed the title Add customization to print events drop them feat: Add customization to print events drop them Nov 29, 2024
@robbie-c robbie-c requested a review from pauldambra November 29, 2024 16:33
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.

💖

@robbie-c robbie-c closed this Nov 29, 2024
@robbie-c robbie-c deleted the feature/add-print-and-drop-all branch November 29, 2024 16:34
@robbie-c robbie-c restored the feature/add-print-and-drop-all branch November 29, 2024 16:34
@robbie-c robbie-c reopened this Nov 29, 2024
@robbie-c
Copy link
Member Author

^ ignore that noise please - I've definitely used a computer before

Copy link

github-actions bot commented Nov 29, 2024

Size Change: +73 B (0%)

Total Size: 3.18 MB

Filename Size Change
dist/customizations.full.js 12.5 kB +73 B (+0.59%)
ℹ️ View Unchanged
Filename Size
dist/all-external-dependencies.js 206 kB
dist/array.full.es5.js 258 kB
dist/array.full.js 361 kB
dist/array.full.no-external.js 360 kB
dist/array.js 175 kB
dist/array.no-external.js 174 kB
dist/dead-clicks-autocapture.js 14.3 kB
dist/exception-autocapture.js 9.37 kB
dist/external-scripts-loader.js 2.29 kB
dist/main.js 176 kB
dist/module.full.js 361 kB
dist/module.full.no-external.js 360 kB
dist/module.js 175 kB
dist/module.no-external.js 174 kB
dist/recorder-v2.js 115 kB
dist/recorder.js 115 kB
dist/surveys-preview.js 57.6 kB
dist/surveys.js 63.2 kB
dist/tracing-headers.js 1.75 kB
dist/web-vitals.js 10.3 kB

compressed-size-action

@robbie-c robbie-c merged commit a151df5 into main Nov 29, 2024
31 checks passed
@robbie-c robbie-c deleted the feature/add-print-and-drop-all branch November 29, 2024 16:43
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.

Option to disable sending events and only log them
2 participants