Financial-Times / origami

The Origami Component System
52 stars 14 forks source link

feat: Add custom params on video #1703

Closed AniaMakes closed 3 weeks ago

AniaMakes commented 3 weeks ago

Describe your changes

The Ads team needs to ensure the video ad requests include all the relevant targeting values in cust_params.

To achieve this, this PR adds a new o-video config object advertCustomParams. It is added in such a way that if that object is not present, there is no cust_params in the advert targeting url, thus this change can be released as a minor version.

This screenshot is an example of this wired up into next-video-page - note the presence of cust_params in the third line of Request URL and details such as guid, gender, slv which are not there at the moment. Screenshot 2024-06-10 at 12 04 35

Issue ticket number and link

https://financialtimes.atlassian.net/browse/ADSDEV-1881

Link to Figma designs

n/a

Checklist before requesting a review