guardian / frontend

The Guardian DotCom.
https://theguardian.com
Other
5.85k stars 555 forks source link

Remove High Merchandising from Targeting section in admin commercial tools #27525

Closed emma-imber closed 1 month ago

emma-imber commented 1 month ago

What does this change?

Removes High merchandising targeting information from the targeting window in the frontend commercial menu. There was quite a bit of underlying logic that we've been able to delete as it was only being used by this targeting information page.

Screenshots

Before

Screenshot 2024-10-10 at 15 11 47

After

Screenshot 2024-10-10 at 15 11 57
github-actions[bot] commented 1 month ago

Deploy build 3090 of dotcom:frontend-all to CODE

All deployment options - [Deploy build 3090 of `dotcom:frontend-all` to CODE](https://riffraff.gutools.co.uk/deployment/deployAgain?project=dotcom%3Afrontend-all&build=3090&stage=CODE&updateStrategy=MostlyHarmless&action=deploy) - [Deploy parts of build 3090 to CODE by previewing it first](https://riffraff.gutools.co.uk/preview/yaml?project=dotcom%3Afrontend-all&build=3090&stage=CODE&updateStrategy=MostlyHarmless) - [What's on CODE right now?](https://riffraff.gutools.co.uk/deployment/history?projectName=dotcom%3Afrontend-all&stage=CODE)

From guardian/actions-riff-raff.

ashishpuliyel commented 1 month ago

I can't pretend I've fully reviewed this. But what a great PR!

prout-bot commented 1 month ago

Seen on ADMIN-PROD (merged by @emma-imber 13 minutes and 2 seconds ago)

prout-bot commented 1 month ago

Seen on FRONTS-PROD (merged by @emma-imber 13 minutes and 7 seconds ago)