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

V7: Rename trackError -> _track and remove from public types #675

Merged
merged 3 commits into from
Dec 18, 2019

Conversation

bengourley
Copy link
Contributor

Sessions are now part of the public interface via onSession callbacks, so the public trackError() function has been suitably renamed (it takes events not errors) and made private.

@bugsnagbot
Copy link
Collaborator

bugsnagbot commented Dec 17, 2019

@bugsnag/browser bundle size diff

Minified Minfied + Gzipped
Before 40.36 kB 12.24 kB
After 40.35 kB 12.24 kB
± -8 bytes -1 bytes

Generated by 🚫 dangerJS against 7b29890

@bengourley
Copy link
Contributor Author

bengourley commented Dec 18, 2019

I can't fathom why this change increased the bundle size 🤔

it's now a tiny bit smaller which is to be expected

packages/core/test/session.test.js Outdated Show resolved Hide resolved
@bengourley bengourley merged commit 117950f into v7 Dec 18, 2019
@bengourley bengourley deleted the v7-track-event branch December 18, 2019 18:26
@bengourley bengourley mentioned this pull request Apr 14, 2020
Merged
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.

3 participants