Merge pull request #19961 from guardian/kc-section-target · devhttps/frontend@a366611 · GitHub

devhttps / frontend Public

forked from guardian/frontend

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Wiki
• Security and quality
• Insights

Additional navigation options

Commit a366611

Browse filesBrowse the repository at this point in the historyBrowse files

kelvin-chappell

authored

Merge pull request guardian#19961 from guardian/kc-section-target

Add section to custom ad targeting

2 parents 0d7b793 + 3096564 commit a366611
Copy full SHA for a366611

1 file changed

+1Lines changed: 1 addition & 0 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• static/src/javascripts/projects/common/modules/commercial
  • build-page-targeting.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎static/src/javascripts/projects/common/modules/commercial/build-page-targeting.js‎

Copy file name to clipboardExpand all lines: static/src/javascripts/projects/common/modules/commercial/build-page-targeting.js

+1Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -186,6 +186,7 @@ const buildPageTargeting = once(
186	186	? (Math.ceil(page.videoDuration / 30.0) * 30).toString()
187	187	: undefined,
188	188	cc: geolocationGetSync(),
	189	+ s: page.section, // for reference in a macro, so cannot be extracted from ad unit
189	190	},
190	191	page.sharedAdTargeting,
191	192	paTargeting,

0 commit comments

Comments

0 (0)