guardian / dotcom-rendering

The Guardian web rendering service (aka DCR/DCAR)
https://www.theguardian.com
Apache License 2.0
253 stars 29 forks source link

Take into account `hasPageSkin` in merchandising high logic #7620

Closed ioannakok closed 1 year ago

ioannakok commented 1 year ago

Once DCR supports Page Skin Ads we need to make sure we're taking this into account in the logic that calculates the merchandising high position. See frontend logic.

rhiannareechaye commented 1 year ago

@arelra fyi

arelra commented 1 year ago

After investigation with @ioannakok we concluded the current Frontend merchandising-high logic does not do anything with the hasPageSkin boolean. This ticket can be closed with no further action required.