search

guardian / frontend

The Guardian DotCom.
https://theguardian.com
Other
5.84k stars 554 forks source link

Seperate FlexibleContainers into their own class #27495

Closed abeddow91 closed 1 week ago

abeddow91 commented 1 week ago

What does this change?

Seperates out the flexible containers into their own classes. Previously, the new flexible general and special containers were a copy and paste of the dynamic containers and extended dynamic containers. This PR cleans up the code leaving only the relevant methods and extracts them into their own classes. This will make the clean up in november easier when old (dynamic) containers start getting deprecated.

github-actions[bot] commented 1 week ago

Deploy build 2969 of dotcom:frontend-all to CODE

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

Seen on FRONTS-PROD, ADMIN-PROD (created by @abeddow91 and merged by @cemms1 24 minutes and 41 seconds ago)