Skip to content

Fix newTracker optional configuration typing - #1237

Merged
Peter Perlepes (igneel64) merged 1 commit into
release/4.0.0from
fix/remove-newTracker-faulty-typing
Oct 13, 2023
Merged

Fix newTracker optional configuration typing#1237
Peter Perlepes (igneel64) merged 1 commit into
release/4.0.0from
fix/remove-newTracker-faulty-typing

Conversation

@igneel64

Copy link
Copy Markdown
Contributor

Remove unneeded empty object on newTracker initialization. There is no reason for this to be there as all the downstream APIs accept configuration as an optional param without an initializer.

@bundlemon

bundlemon Bot commented Sep 13, 2023

Copy link
Copy Markdown

BundleMon

Files added (6)
Status Path Size Limits
libraries/browser-tracker-core/dist/index.mod
ule.js
+25.34KB 26KB / +10%
trackers/javascript-tracker/dist/sp.js
+24.28KB 25KB / +10%
trackers/javascript-tracker/dist/sp.lite.js
+14.58KB 15KB / +10%
trackers/browser-tracker/dist/index.umd.min.j
s
+14.42KB 15KB / +10%
libraries/tracker-core/dist/index.module.js
+13.36KB 15KB / +10%
trackers/browser-tracker/dist/index.module.js
+3.49KB 5KB / +10%

Total files change +95.46KB 0%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants