frontend icon indicating copy to clipboard operation
frontend copied to clipboard

Labs front inconsitent behaviour when `adFree`

Open ioannakok opened this issue 2 years ago • 7 comments

I've been experiencing an issue today and cannot figure out why. This link https://www.theguardian.com/guardian-labs-australia is showing two different fronts for me. The right one, when I am not logged in to the Guardian site and when I am logged in, it shows a stale front (articles from 2018).

ioannakok avatar Jun 13 '22 09:06 ioannakok

https://user-images.githubusercontent.com/19683595/173323675-7dee4d03-4a73-4237-baa5-e6e93afaba22.mov

ioannakok avatar Jun 13 '22 09:06 ioannakok

This looks like a bug, as the following are working as expected even if signed-in:

  • https://www.theguardian.com/guardian-labs
  • https://www.theguardian.com/guardian-labs-us

comparing guardian labs

mxdvl avatar Jun 13 '22 09:06 mxdvl

Ongoing research to identify why the paid containers are skipped for this page. Potential areas of interest are :

  • https://github.com/guardian/frontend/blob/main/common/app/views/fragments/containers/facia_cards/container.scala.html#L30
  • https://github.com/guardian/frontend/blob/main/common/app/layout/FaciaContainer.scala#L163-L166
  • https://github.com/guardian/frontend/blob/main/common/app/model/facia/PressedCollection.scala#L37-L43

mxdvl avatar Jun 13 '22 16:06 mxdvl

@mxdvl Please leave a comment if there's anything else to add. Putting this back to TODO

ioannakok avatar Jun 17 '22 09:06 ioannakok

Picking this up to try and figure it out!

OllysCoding avatar Jun 17 '22 10:06 OllysCoding

I believe the start of the issue could stem from here: https://github.com/guardian/frontend/blob/main/facia/app/controllers/FaciaController.scala#L194

This logic says that if there are no collections in the 'ad free' variant of a front (which is prioritised for an ad-free user), then fall back to the normal variant.

The ad-free https://www.theguardian.com/guardian-labs-us has no collections:

Image

Whereas the aus ad-free version does:

Image

We can also see that the ad-free variant of the aus version is still being pressed by the updated time:

Image

So it's likely that either in the configuration, history, or pressing of the article, something is wrong or misconfigured that's causing the ad-free variant of the article to be pressed with content we don't want.

OllysCoding avatar Jun 17 '22 15:06 OllysCoding

Hi all, has this been merged/is there an associated PR?

jlieb10 avatar Jul 13 '22 14:07 jlieb10

this is in the In Progress column in the Rota board -- is it still in progress? If not, does it need to be picked up again?

cc. @ioannakok @OllysCoding

bryophyta avatar Nov 17 '22 16:11 bryophyta