Skip to content

[AE-1184] Add contextual targeting metrics#11

Merged
abedef merged 2 commits intomainfrom
ae-1184-contextual-targeting-info
Mar 4, 2026
Merged

[AE-1184] Add contextual targeting metrics#11
abedef merged 2 commits intomainfrom
ae-1184-contextual-targeting-info

Conversation

@abedef
Copy link
Copy Markdown
Contributor

@abedef abedef commented Feb 17, 2026

This PR adds two metrics, taxonomy and categories, to support contextual targeting for direct sold stories (ticket).

Measurement Description Data Collection Category Tracking Bug #
ad.taxonomy is a taxonomy identifier for the contextual categories (e.g. "IAB-3.0"). May be null if no contextual information was provided. Category 1 https://bugzilla.mozilla.org/show_bug.cgi?id=2017204
ad.categories is a comma-separated list of contextual category identifiers from the taxonomy (e.g. "210,315"). May be null if no contextual information was provided. Category 2 https://bugzilla.mozilla.org/show_bug.cgi?id=2017204

@abedef abedef force-pushed the ae-1184-contextual-targeting-info branch from d8b7b26 to a034047 Compare February 19, 2026 16:15
Copy link
Copy Markdown
Contributor

@mashalifshin mashalifshin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Total nitpick on the PR title, but could you please reword to something like "Add contextual targeting metrics"? (this PR adds metrics to existing pings, but title kinda sounds like you're adding new pings)

Comment thread telemetry/glean/metrics.yaml Outdated
Comment thread telemetry/glean/metrics.yaml Outdated
Comment thread telemetry/glean/metrics.yaml Outdated
@abedef abedef changed the title [AE-1184] Add contextual targeting pings [AE-1184] Add contextual targeting metrics Feb 19, 2026
@mashalifshin
Copy link
Copy Markdown
Contributor

@abedef I met with Nick Kypreos today to talk through the "live test" plan and he was strongly in favor of including the taxonomy column in the first pass for future-proofing version changes, even if it would just store the same value. So I think since Glenda confirmed there's no additional cost here, let's leave it.

@abedef abedef marked this pull request as ready for review March 4, 2026 19:56
@abedef abedef requested a review from a team as a code owner March 4, 2026 19:56
Copy link
Copy Markdown
Contributor

@mashalifshin mashalifshin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR has gone through the data steward review process, and the sensitive data review process, and received approval, so now I can go ahead and Approve this PR.

See links in the PR description for data review details.

@abedef abedef merged commit 398d717 into main Mar 4, 2026
2 checks passed
@abedef abedef deleted the ae-1184-contextual-targeting-info branch March 4, 2026 23:02
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.

4 participants