Merge pull request #14369 from guardian/doml/opinion-avatar

Create "OpinionNoAvatar" AB test

Author: domlander

dotcom-rendering/src/components/DecideContainer.tsx

@@ -47,6 +47,7 @@ type Props = {
 	frontId?: string;
 	collectionId: number;
 	containerLevel?: DCRContainerLevel;
+	isInOpinionNoAvatarVariant?: boolean;
 };
 
 export const DecideContainer = ({
@@ -62,6 +63,7 @@ export const DecideContainer = ({
 	frontId,
 	collectionId,
 	containerLevel,
+	isInOpinionNoAvatarVariant,
 }: Props) => {
 	switch (containerType) {
 		case 'dynamic/fast':
@@ -257,6 +259,7 @@ export const DecideContainer = ({
 					aspectRatio={aspectRatio}
 					containerLevel={containerLevel}
 					collectionId={collectionId}
+					isInOpinionNoAvatarVariant={isInOpinionNoAvatarVariant}
 				/>
 			);
 		case 'scrollable/small':
@@ -286,6 +289,7 @@ export const DecideContainer = ({
 						absoluteServerTimes={absoluteServerTimes}
 						aspectRatio={aspectRatio}
 						sectionId={sectionId}
+						isInOpinionNoAvatarVariant={isInOpinionNoAvatarVariant}
 					/>
 				</Island>
 			);

dotcom-rendering/src/components/FlexibleGeneral.tsx

@@ -32,6 +32,7 @@ type Props = {
 	aspectRatio: AspectRatio;
 	containerLevel?: DCRContainerLevel;
 	collectionId: number;
+	isInOpinionNoAvatarVariant?: boolean;
 };
 
 type RowLayout = 'oneCardHalfWidth' | 'oneCardFullWidth' | 'twoCard';
@@ -581,6 +582,7 @@ export const FlexibleGeneral = ({
 	aspectRatio,
 	containerLevel = 'Primary',
 	collectionId,
+	isInOpinionNoAvatarVariant,
 }: Props) => {
 	const splash = [...groupedTrails.splash].slice(0, 1).map((snap) => ({
 		...snap,
@@ -592,6 +594,9 @@ export const FlexibleGeneral = ({
 		.map((standard, i) => ({
 			...standard,
 			uniqueId: `collection-${collectionId}-standard-${i}`,
+			avatarUrl: isInOpinionNoAvatarVariant
+				? undefined
+				: standard.avatarUrl,
 		}));
 
 	const groupedCards = decideCardPositions(cards);

dotcom-rendering/src/components/ScrollableMedium.importable.tsx

@@ -17,6 +17,7 @@ type Props = {
 	containerType: DCRContainerType;
 	aspectRatio: AspectRatio;
 	sectionId: string;
+	isInOpinionNoAvatarVariant?: boolean;
 };
 
 /**
@@ -35,6 +36,7 @@ export const ScrollableMedium = ({
 	showAge,
 	aspectRatio,
 	sectionId,
+	isInOpinionNoAvatarVariant,
 }: Props) => {
 	return (
 		<ScrollableCarousel
@@ -51,7 +53,12 @@ export const ScrollableMedium = ({
 				return (
 					<ScrollableCarousel.Item key={trail.url}>
 						<FrontCard
-							trail={trail}
+							trail={{
+								...trail,
+								avatarUrl: isInOpinionNoAvatarVariant
+									? undefined
+									: trail.avatarUrl,
+							}}
 							imageLoading={imageLoading}
 							absoluteServerTimes={!!absoluteServerTimes}
 							containerPalette={containerPalette}

dotcom-rendering/src/layouts/FrontLayout.tsx

@@ -136,6 +136,15 @@ export const FrontLayout = ({ front, NAV }: Props) => {
 
 	const contributionsServiceUrl = getContributionsServiceUrl(front);
 
+	/**
+	 * We are running an AB test which replaces the avatar for the card image
+	 * in the Opinion and More opinion collections on network fronts.
+	 */
+	const isInOpinionNoAvatarVariant = (collectionName: string) =>
+		abTests.opinionNoAvatarVariant === 'variant' &&
+		front.isNetworkFront &&
+		(collectionName === 'Opinion' || collectionName === 'More opinion');
+
 	const fallbackAspectRatio = (collectionType: DCRContainerType) => {
 		switch (collectionType) {
 			case 'scrollable/feature':
@@ -634,6 +643,9 @@ export const FrontLayout = ({ front, NAV }: Props) => {
 									sectionId={ophanName}
 									collectionId={index + 1}
 									containerLevel={collection.containerLevel}
+									isInOpinionNoAvatarVariant={isInOpinionNoAvatarVariant(
+										collection.displayName,
+									)}
 								/>
 							</FrontSection>