chore(glean): add page's UTM parameters to pings #9595
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
https://mozilla-hub.atlassian.net/browse/MP-545
https://bugzilla.mozilla.org/show_bug.cgi?id=1851150
Problem
We can't easily attribute traffic which adds utm parameters through Glean.
Solution
Add utm parameters to our Glean pings.
How did you test this change?
yarn && yarn dev
@fiji-flo after sending this ping, I realised we already sorta send utm parameters to glean, through the
page.path
metric - since we'll now be sending these in a structured form, should I wipeutm_
parameters out of the url we send in that metric (or even not send any query parameters inpage.path
at all, to keep it "clean"?)