guardian / frontend

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

Support DCR for apps video article requests #27542

Closed arelra closed 1 month ago

arelra commented 1 month ago

What does this change?

Support 'DCR for apps' video article requests e.g.

https://www.theguardian.com/uk-news/video/2024/oct/21/king-charles-and-queen-camilla-on-tour-in-canberra-video?dcr=apps

Currently such requests to Frontend do not respect the ?dcr=apps url parameter.

Therefore Frontend does not send an apps version of the request to DCR.

This change adds handling for a 'DCR for apps' video article request.

Screenshots

Before After
before after

Checklist

github-actions[bot] commented 1 month ago

Deploy build 3138 of dotcom:frontend-all to CODE

All deployment options - [Deploy build 3138 of `dotcom:frontend-all` to CODE](https://riffraff.gutools.co.uk/deployment/deployAgain?project=dotcom%3Afrontend-all&build=3138&stage=CODE&updateStrategy=MostlyHarmless&action=deploy) - [Deploy parts of build 3138 to CODE by previewing it first](https://riffraff.gutools.co.uk/preview/yaml?project=dotcom%3Afrontend-all&build=3138&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.

prout-bot commented 1 month ago

Seen on ADMIN-PROD (merged by @arelra 12 minutes and 5 seconds ago)

prout-bot commented 1 month ago

Seen on FRONTS-PROD (merged by @arelra 13 minutes and 11 seconds ago)