Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement audio teaser view tracking in n-tracking #21

Open
magsallen opened this issue Aug 6, 2019 · 1 comment
Open

Implement audio teaser view tracking in n-tracking #21

magsallen opened this issue Aug 6, 2019 · 1 comment

Comments

@magsallen
Copy link
Contributor

magsallen commented Aug 6, 2019

Implement audio teaser view tracking in n-tracking as it's required by several FT applications.

This code is currently required in the search page, the article page and the stream page. As the audio team is currently winding down, it makes sense to centralise this code so that the tracking events being sent by each application remain consistent without the active oversight of a team.

Example implementation from https://github.com/Financial-Times/next-search-page/pull/264

oTracking.view.init({
	selector: '.o-teaser--audio',
	getContextData: (el) => {
		return {
			componentContentId: el.getAttribute('data-id'),
			component: 'teaser',
			type: 'audio',
			subtype: 'podcast'
		};
	}
});```
@i-like-robots
Copy link
Contributor

i-like-robots commented Aug 6, 2019

I agree with the rationale, but I don't think should be as straightforward as copy and pasting it.

In order to ensure this package remains supportable we need to attach some extra information to it:

  • Which developers know about it?
  • Who owns it?
  • Which tracking spec does this implement?
  • Is there a Chartio dashboard associated with it?
  • Is there a known expiry date? If not, can we add one?

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

No branches or pull requests

2 participants