Merge pull request #32 from Financial-Times/CI-1165-replace-podcast-d…

…escription-with-standfirst Ci 1165 replace podcast description with standfirst
Financial-Times · Oct 13, 2022 · ef9d680 · ef9d680
2 parents a4e20d9 + 816b824
commit ef9d680
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/lib/get-topper-settings.js b/src/lib/get-topper-settings.js
@@ -130,7 +130,7 @@ const usePodcastTopper = (content, flags) => {
 		includesImage: true,
 		fthead: content.mainImage && content.mainImage.url,
 		modifiers: ['branded', 'has-headshot'],
-		standfirst: content.byline,
+		standfirst: content.standfirst,
 		myFtButton: {
 			variant: 'inverse',
 			followPlusDigestEmail: followPlusDigestEmail(flags)

diff --git a/test/lib/get-topper-settings.test.js b/test/lib/get-topper-settings.test.js
@@ -119,7 +119,7 @@ describe('Get topper settings', () => {
 		it('returns the podcast topper', () => {
 			const topper = getTopperSettings({
 				subtype: 'podcast',
-				byline: 'byline',
+				standfirst: 'example-standfirst',
 				mainImage: {
 					url: 'mainImageUrl'
 				}
@@ -128,7 +128,7 @@ describe('Get topper settings', () => {
 			expect(topper).to.deep.include({
 				isPodcast: true,
 				fthead: 'mainImageUrl',
-				standfirst: 'byline'
+				standfirst: 'example-standfirst'
 			});
 		});
 	});