Merge pull request #13391 from guardian/ei/add-comscorekw-comment

Add comscorekw as comment to html source

Author: emma-imber

dotcom-rendering/src/server/htmlPageTemplate.ts

@@ -30,7 +30,7 @@ type BaseProps = {
 
 interface WebProps extends BaseProps {
 	renderingTarget: 'Web';
-	keywords: string;
+	section: string;
 	config: Config & { renderingTarget: 'Web' };
 }
 
@@ -363,14 +363,15 @@ https://workforus.theguardian.com/careers/product-engineering/
 					renderingTarget === 'Web'
 						? `
                 <noscript>
+					<!-- Comscore Identifier: comscorekw=${props.section} -->
                     <img src="https://sb.scorecardresearch.com/p?${new URLSearchParams(
 						{
 							c1: '2',
 							c2: '6035250',
 							cv: '2.0',
 							cj: '1',
 							cs_ucfr: '0',
-							comscorekw: props.keywords,
+							comscorekw: props.section,
 						},
 					).toString()}" />
                 </noscript>

dotcom-rendering/src/server/render.allEditorialNewslettersPage.web.tsx

@@ -95,7 +95,7 @@ export const renderEditorialNewslettersPage = ({
 		title,
 		description: newslettersPage.description,
 		guardian,
-		keywords: '',
+		section: '',
 		renderingTarget: 'Web',
 		weAreHiring: !!newslettersPage.config.switches.weAreHiring,
 		config,

dotcom-rendering/src/server/render.article.web.tsx

@@ -178,12 +178,7 @@ export const renderHtml = ({
 	const ampLink = getAmpLink(frontendData.tags);
 
 	const { openGraphData, twitterData } = frontendData;
-	const keywords =
-		typeof frontendData.config.keywords === 'undefined' ||
-		frontendData.config.keywords === 'Network Front'
-			? ''
-			: frontendData.config.keywords;
-
+	const section = frontendData.config.section;
 	const initTwitter = `
 <script>
 // https://developer.twitter.com/en/docs/twitter-for-websites/javascript-api/guides/set-up-twitter-for-websites
@@ -218,7 +213,7 @@ window.twttr = (function(d, s, id) {
 		ampLink,
 		openGraphData,
 		twitterData,
-		keywords,
+		section,
 		initTwitter:
 			pageHasTweetElements || format.design === ArticleDesign.LiveBlog
 				? initTwitter

dotcom-rendering/src/server/render.footballDataPage.web.tsx

@@ -134,7 +134,7 @@ export const renderFootballDataPage = (footballData: DCRFootballDataPage) => {
 			googleRecaptchaSiteKey: footballData.config.googleRecaptchaSiteKey,
 			unknownConfig: footballData.config,
 		}),
-		keywords: '',
+		section: '',
 		renderingTarget,
 		weAreHiring: !!footballData.config.switches.weAreHiring,
 		config,

dotcom-rendering/src/server/render.front.web.tsx

@@ -146,7 +146,7 @@ export const renderFront = ({
 		unknownConfig: front.config,
 	});
 
-	const keywords = front.config.keywords;
+	const section = front.config.section;
 
 	const canonicalUrl =
 		front.isNetworkFront &&
@@ -162,7 +162,7 @@ export const renderFront = ({
 		title,
 		description: front.pressedPage.seoData.description,
 		guardian,
-		keywords,
+		section,
 		renderingTarget: 'Web',
 		hasPageSkin: front.config.hasPageSkin,
 		weAreHiring: !!front.config.switches.weAreHiring,
@@ -248,7 +248,7 @@ export const renderTagPage = ({
 		unknownConfig: tagPage.config,
 	});
 
-	const keywords = tagPage.config.keywords;
+	const section = tagPage.config.section;
 
 	const pageHtml = htmlPageTemplate({
 		scriptTags,
@@ -257,7 +257,7 @@ export const renderTagPage = ({
 		title,
 		description: tagPage.header.description,
 		guardian,
-		keywords,
+		section,
 		renderingTarget: 'Web',
 		weAreHiring: !!tagPage.config.switches.weAreHiring,
 		canonicalUrl: tagPage.canonicalUrl,