ad server handle shims for 2 new events
Categories
(Firefox :: New Tab Page, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | verified |
People
(Reporter: thecount, Assigned: thecount)
References
Details
(Keywords: github-merged)
Attachments
(2 files)
52 bytes,
text/x-github-pull-request
|
Details | Review | |
2.38 KB,
text/plain
|
klong
:
data-review+
|
Details |
Right now we have a shim for clicks and impressions, see bugs 1558903 and 1550119.
We want to expand on this with shims for existing events for saving a story an dismissing a story.
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 1•5 years ago
|
||
Ideally: get this landed before code freeze (6/28). QA may have to happen for this in Beta, or during code freeze week in Nightly.
Comment 2•5 years ago
|
||
Assignee | ||
Comment 3•5 years ago
|
||
Data review for spoc shim, pretty much the same deal as the last one, but instead of click and view pings, this is for dismiss and save to pocket pings
Comment 5•5 years ago
|
||
HI Kenny,
The data review info in Comment 3 says:
- Why answer these?
AdZerk will use the shim to control the speed at which ads are delivered (“pacing”) and prevent users from seeing ads more than the allowed limit.
But the description above says the shim is now looking to be extended to the save/dismiss interaction. Is the data review in Comment 3 just for reference? Looking to make sure there's not a disconnect in the request.
The save/dismiss interactions are used by AdZerk to prevent users from seeing ads they should not (ads they have dismissed or saved), so I think Scott and I are on the same page for this data request/review... But is there something else you see that doesn't line up?
Comment 7•5 years ago
•
|
||
It looked like the use of the shim for pacing (from the data review info in Comment 3) was different than it's use as described for this request for save/dismiss interactions so I just wanted to make sure I wasn't misunderstanding the ask.
I don't have any concerns about using the shim for save/dismiss; as with its use for the clicks and impressions there's no change to the basic properties of the new tab collection.
Let's make sure that these new events also are accounted for when the user deletes their data (see treatment in Bug 1553933). It should since its SPOC info associated with their ID.
Assignee | ||
Comment 8•5 years ago
|
||
Yup that all makes sense to me. :thumbsup:
Comment 10•5 years ago
|
||
Comment 11•5 years ago
|
||
I have verified that the save and delete shims are sent based on the action taken by the user (Save to Pocket, Dismiss) on the latest Nightly 69.0a1 (Build ID 20190618214619) on Windows 10, macOS 10.14, and Arch Linux 4.14.3.
Updated•5 years ago
|
Description
•