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

Provide more data in Adhese bid response #5154

Merged
merged 13 commits into from
Apr 28, 2020
Merged

Conversation

kimvc
Copy link
Contributor

@kimvc kimvc commented Apr 22, 2020

Type of change

  • [x ] Feature
  • [ x] Other

Description of change

  • Update to provide more Adhese related data in the bid response. Doesn't have any affect on running implementations.

  • Updated the tests to reflect the changes in the adapter.

  • contact email of the adapter’s maintainer: support@adhese.eu

  • official adapter submission

Other information

@kimvc
Copy link
Contributor Author

kimvc commented Apr 28, 2020

Hi, is there any reason why the review is taking longer than expected?
Is it related to the failed CircleCI test? Because that doesn't seem related to our changes:

1) Returns valid data if array of bids is valid
     ColossussspAdapter buildRequests with user ids
     ReferenceError: 'Symbol' is undefined
   at Anonymous function (webpack:///test/spec/modules/colossussspBidAdapter_spec.js:122:9 <- test/test_index.js:113594:16)

Copy link
Contributor

@sumit116 sumit116 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@sumit116 sumit116 added the LGTM label Apr 28, 2020
@sumit116 sumit116 merged commit 2cfcf9c into prebid:master Apr 28, 2020
jimee02 pushed a commit to openx/Prebid.js that referenced this pull request Jun 12, 2020
* Add support for bidderRequest.refererInfo in Adhese Adapter.

* Add support for bidderRequest.refererInfo in Adhese Adapter.

* Jira AD-2462 / add tlall is consent string is present

* Added 'adhese' attribute to bid that contains meta data - Jira AD-2642

* added DALE to adhese determination

* extra config option: no format, but use size array as format string

* Added more values to originData

* Revert "Added more values to originData"

This reverts commit 57f0033.

* Pulled Sander's changes & added more data to originData

* Adhese bid adapter - final version

Co-authored-by: Mateusz Michalowski <mateusz@adhese.eu>
Co-authored-by: Mateusz Michalowski <mateusz@michalowski.biz>
Co-authored-by: Tim Sturtewagen <tim@adhese.com>
Co-authored-by: Kim Van Crombrugge <kim@adhese.eu>
Co-authored-by: Sander <sander@adhese.eu>
iggyfisk pushed a commit to happypancake/Prebid.js that referenced this pull request Jun 22, 2020
* Add support for bidderRequest.refererInfo in Adhese Adapter.

* Add support for bidderRequest.refererInfo in Adhese Adapter.

* Jira AD-2462 / add tlall is consent string is present

* Added 'adhese' attribute to bid that contains meta data - Jira AD-2642

* added DALE to adhese determination

* extra config option: no format, but use size array as format string

* Added more values to originData

* Revert "Added more values to originData"

This reverts commit 57f0033.

* Pulled Sander's changes & added more data to originData

* Adhese bid adapter - final version

Co-authored-by: Mateusz Michalowski <mateusz@adhese.eu>
Co-authored-by: Mateusz Michalowski <mateusz@michalowski.biz>
Co-authored-by: Tim Sturtewagen <tim@adhese.com>
Co-authored-by: Kim Van Crombrugge <kim@adhese.eu>
Co-authored-by: Sander <sander@adhese.eu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants