Conversation
d8b7b26 to
a034047
Compare
mashalifshin
left a comment
There was a problem hiding this comment.
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)
|
@abedef I met with Nick Kypreos today to talk through the "live test" plan and he was strongly in favor of including the |
mashalifshin
left a comment
There was a problem hiding this comment.
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.
This PR adds two metrics,
taxonomyandcategories, to support contextual targeting for direct sold stories (ticket).ad.taxonomyis a taxonomy identifier for the contextual categories (e.g. "IAB-3.0"). May be null if no contextual information was provided.ad.categoriesis a comma-separated list of contextual category identifiers from the taxonomy (e.g. "210,315"). May be null if no contextual information was provided.