Stats: Fix tracking of search terms, tags & parsing of tax queries#43879
Stats: Fix tracking of search terms, tags & parsing of tax queries#43879
Conversation
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 Follow this PR Review Process:
If you have questions about anything, reach out in #jetpack-developers for guidance! |
Code Coverage SummaryCoverage changed in 1 file.
|
myhro
left a comment
There was a problem hiding this comment.
LGTM. ✅
Tried the following cases:
Search
v: ext
blog: 236297789
post: 0
tz: 0
srv: jetpack.myhrowp.com
arch_search: lorem
arch_filters: posts_per_page=10&paged=1&orderby=&order=DESC
arch_results: 1
j: 1:14.8-a.3
host: jetpack.myhrowp.com
ref: https://jetpack.myhrowp.com/
fcp: 301
rand: 0.17667270060733986Category
v: ext
blog: 236297789
post: 0
tz: 0
srv: jetpack.myhrowp.com
arch_cat: writings
arch_results: 2
j: 1:14.8-a.3
host: jetpack.myhrowp.com
ref: https://jetpack.myhrowp.com/2025/03/12/lorem-ipsum/
fcp: 279
rand: 0.8811503500882137Tag
v: ext
blog: 245393704
post: 0
tz: 0
srv: remaining-crane-equinox.jurassic.ninja
arch_tag: tip
arch_results: 1
hp: atomic
ac: 32
amp: 0
j: 1:14.8-a.4
host: remaining-crane-equinox.jurassic.ninja
ref
rand: 0.25471570133750665Taxonomy
v: ext
blog: 245393704
post: 0
tz: 0
srv: remaining-crane-equinox.jurassic.ninja
arch_tax_help: intro-101
arch_results: 1
hp: atomic
ac: 32
amp: 0
j: 1:14.8-a.4
host: remaining-crane-equinox.jurassic.ninja
ref
rand: 0.08679834836917522|
@sotirispl I'm trying to fix the labels based on GitHub Actions bot suggestions:
I've left the |
aede6a9 to
35a90ef
Compare

Fixes
Proposed changes:
queryandquery_varsarrays.Other information:
Jetpack product discussion
Does this pull request change what data or activity we track or use?
Testing instructions:
sURL parameter and another that makes a taxonomy query too (e.g.?s=searchterm&testtax=taxonomy) and verify that both pixel requests contain the expected payload.