1456145 - Add impression ping for AS router snippets

Status: RESOLVED FIXED

(Firefox :: Messaging System, enhancement, P1)

Comment 1

[Kate Hudson :k88hudson]

We need an impression ping for AS router templates. It should:

- only be sent when the AS router is actually visible
- have a new helper method (see https://github.com/mozilla/activity-stream/blob/4d6cf505c48643b4ad8306d9c66c0a3d320a65a5/system-addon/content-src/message-center/message-center-content.jsx#L30) that defines a ping action type (should be something like ${PROVIDER}_impression) 
- can borrow from code that already exists to track impression pings in AS

:nanj can help handle this ping / define what the ping action should be

Comment 2

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/activity-stream/pull/4127

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/bdf9ca55d3520d143964b44612d54eeb9e663e04
Bug 1456145 - Add impression ping for AS Router

https://github.com/mozilla/activity-stream/commit/2f718eb067433c36b68d84727b454b5f6ac3e3be
Bug 1456145 - Follow-up on impression pings for AS Router

Comment 4

[Ed Lee :Mardak]

https://hg.mozilla.org/mozilla-central/rev/a9f5a8dc2e22