Skip to content

Commit

Permalink
Merge pull request #12413 from guardian/dina/remove-survey-switch
Browse files Browse the repository at this point in the history
Remove `surveys` switch
  • Loading branch information
deedeeh committed Sep 19, 2024
2 parents 4d606c3 + bb44b81 commit f53da77
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 10 deletions.
11 changes: 3 additions & 8 deletions dotcom-rendering/src/layouts/InteractiveLayout.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -300,14 +300,9 @@ export const InteractiveLayout = (props: WebProps | AppsProps) => {
</Stuck>
)}

{renderAds &&
article.config.switches.surveys &&
hasSurveyAd && (
<AdSlot
position="survey"
display={format.display}
/>
)}
{renderAds && hasSurveyAd && (
<AdSlot position="survey" display={format.display} />
)}
</>
)}
<main data-layout="InteractiveLayout">
Expand Down
2 changes: 1 addition & 1 deletion dotcom-rendering/src/layouts/NewsletterSignupLayout.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,7 @@ export const NewsletterSignupLayout = ({ article, NAV, format }: Props) => {
/>
</div>

{renderAds && !!article.config.switches.surveys && hasSurveyAd && (
{renderAds && hasSurveyAd && (
<AdSlot position="survey" display={format.display} />
)}

Expand Down
2 changes: 1 addition & 1 deletion dotcom-rendering/src/layouts/StandardLayout.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -461,7 +461,7 @@ export const StandardLayout = (props: WebProps | AppProps) => {
</Stuck>
)}

{renderAds && article.config.switches.surveys && hasSurveyAd && (
{renderAds && hasSurveyAd && (
<AdSlot position="survey" display={format.display} />
)}

Expand Down

0 comments on commit f53da77

Please sign in to comment.