Skip to content

AdYouLike Bidder - Add size in video bid#8282

Merged
ChrisHuie merged 48 commits intoprebid:masterfrom
Adyoulike:master
Apr 14, 2022
Merged

AdYouLike Bidder - Add size in video bid#8282
ChrisHuie merged 48 commits intoprebid:masterfrom
Adyoulike:master

Conversation

@guiann
Copy link
Contributor

@guiann guiann commented Apr 13, 2022

Type of change

  • Feature

Description of change

Some publisher with outstream video renderer are expecting video size to be given as width/height in bid object.

[*] official adapter submission

guiann added 30 commits April 14, 2021 15:23
@ChrisHuie ChrisHuie self-requested a review April 13, 2022 20:35
@ChrisHuie ChrisHuie self-assigned this Apr 13, 2022
@guiann
Copy link
Contributor Author

guiann commented Apr 14, 2022

Tests are unstable, doesn't seem related to our changes, does it?

@ChrisHuie ChrisHuie merged commit 2d5be3f into prebid:master Apr 14, 2022
JoelPM pushed a commit to JoelPM/Prebid.js that referenced this pull request Apr 25, 2022
* add required clickurl in every native adrequest

* allows the native response to be given as is to prebid if possible

* add unit tests on new Native case

* Handle meta object in bid response with default addomains array

* fix icon retrieval in Native case

* Update priorities in case of multiple mediatypes given

* improve robustness and fix associated unit test on picture urls

* add support for params.size parameter

* add unit test on new size format

* Makes sure the playerSize format is consistent

* enable Vast response on bidder adapter

* fix lint errors

* add test on Vast format case

* add userId to bidrequest

* revert package-lock.json changes

* improve multiple mediatype handling

* Expose adyoulike GVL id

* fix icurl issue when retreiving icon for Native mediatype

* update unit tests on icon url in native mediatype

* target video endpoint when video mediatype is present

* add unit test on video endpoint

* detect if bid request has video

* remove console log

* Add size information in Video bid + unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments