Skip to content

Add plugin to send requests with user ID to a Kantar FocalMeter endpoint (close #1133) - #1243

Merged
Matus Tomlein (matus-tomlein) merged 3 commits into
release/3.16.0from
issue/1133-focalmeter_plugin
Oct 5, 2023
Merged

Add plugin to send requests with user ID to a Kantar FocalMeter endpoint (close #1133)#1243
Matus Tomlein (matus-tomlein) merged 3 commits into
release/3.16.0from
issue/1133-focalmeter_plugin

Conversation

@matus-tomlein

Copy link
Copy Markdown
Contributor

Issue #1133

This PR has already been partly reviewed in #1134. Since then we have published it as a pre-release for external testing, now the testing has confirmed that we can release it in a stable tracker release.

Since the reviewed PR, there has been one small change – we have made it possible to pass processUserId callback in the plugin configuration that enables users to apply custom hashing on top of the user identifier.

Documentation for this is here.

…int (close #1133)

PR #1134
# Conflicts:
#	common/config/rush/pnpm-lock.yaml
#	common/config/rush/repo-state.json
#	rush.json

# Conflicts:
#	common/config/rush/pnpm-lock.yaml
#	common/config/rush/repo-state.json
#	rush.json
@bundlemon

bundlemon Bot commented Oct 5, 2023

Copy link
Copy Markdown

BundleMon

Files added (6)
Status Path Size Limits
libraries/browser-tracker-core/dist/index.mod
ule.js
+25.32KB 26KB / +10%
trackers/javascript-tracker/dist/sp.js
+24.18KB 25KB / +10%
trackers/javascript-tracker/dist/sp.lite.js
+14.86KB 15KB / +10%
trackers/browser-tracker/dist/index.umd.min.j
s
+14.7KB 15KB / +10%
libraries/tracker-core/dist/index.module.js
+13.35KB 15KB / +10%
trackers/browser-tracker/dist/index.module.js
+3.51KB 5KB / +10%

Total files change +95.92KB 0%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history

@matus-tomlein
Matus Tomlein (matus-tomlein) merged commit bb28121 into release/3.16.0 Oct 5, 2023
@matus-tomlein
Matus Tomlein (matus-tomlein) deleted the issue/1133-focalmeter_plugin branch October 5, 2023 12:15
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