Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AdYouLike Bidder - enable Vast response format #7229

Merged
merged 26 commits into from
Jul 28, 2021

Conversation

guiann
Copy link
Contributor

@guiann guiann commented Jul 28, 2021

Type of change

Refactoring our Video ad response format.

Description of change

We were testing pure Vast response on our side.
This minor change now enables it it production.

guillaume.andouard@gmail.com

  • [*] official adapter submission

@guiann guiann changed the title AdYouLike Bidder - video response format AdYouLike Bidder - enable Vast response format Jul 28, 2021
@ChrisHuie ChrisHuie self-requested a review July 28, 2021 15:50
@ChrisHuie ChrisHuie self-assigned this Jul 28, 2021
@ChrisHuie
Copy link
Collaborator

@guiann can you please add a test for this case

@guiann
Copy link
Contributor Author

guiann commented Jul 28, 2021

Done. Thx Chris.

@ChrisHuie ChrisHuie merged commit 9d7fdc8 into prebid:master Jul 28, 2021
prebidtappx pushed a commit to prebidtappx/Prebid.js that referenced this pull request Nov 15, 2021
* 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
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