Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Ads for Tag Fronts & respect speed (#8056)
* feat: Add support for tuples Co-Authored-By: Max Duval <[email protected]> * feat: Add support for fast & slow containers on tag fronts * feat: Inject MPU states into containers on Tag fronts Revert "feat: Inject MPU states into containers on Tag fronts" This reverts commit 4d8140dd3ae5e31ebbaa340b332862e1cf5c68ab. Revert "Revert "feat: Inject MPU states into containers on Tag fronts"" This reverts commit 7bca0a66bd23c90d7a4b1c129c72959245d16e51. * WIP: Add support for MPU injection & ads in tag fronts * feat: Add support for desktop & mobile ads on tag fronts * Update slow MPU containers to match frontend for tag fronts * use containerId for key on tag fronts map * Add stories for mpu containers on tag fronts & fix any bugs * Add stories for DecideContainerByTrails & fix layout bugs * refactor: tidier definitions for tuples Co-Authored-By: Max Duval <[email protected]> * refactor: Use exhaustive checks for tag front MPU components Co-Authored-By: Max Duval <[email protected]> * fix: remove unused import * feat: Introduce 'TupleOrLess`and `takeFirst` Co-Authored-By: Max Duval <[email protected]> * refactor: Use exhaustive checking for DecideContainerByTrails Co-Authored-By: Max Duval <[email protected]> * fix: dont render empty ULs if theres only 8 cards * Add tests for tuples takeFirst * chore: Update comment for takeFirst * fix: use correct %'s on DecideContainerByTrails * fix: remove exports from tag front mpu containers as they're not used * Update dotcom-rendering/src/lib/getAdPositions.ts Co-authored-by: Max Duval <[email protected]> * Update dotcom-rendering/src/lib/getAdPositions.ts Co-authored-by: Max Duval <[email protected]> * Merge branch 'olly/tag-fronts-containers-and-ads' of github.com:guardian/dotcom-rendering into olly/tag-fronts-containers-and-ads * chore: remove isTupleOrGreater as this is no longer used * refactor: use takeFirst in injectMpuIntoGroupedTrails * Final tidy-up --------- Co-authored-by: Max Duval <[email protected]> Co-authored-by: Max Duval <[email protected]>
- Loading branch information