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

Interactive Offers Bid Adapter: added data to the final response object #6648

Closed
wants to merge 31 commits into from
Closed

Conversation

IOTiagoFaria
Copy link
Contributor

@IOTiagoFaria IOTiagoFaria commented Apr 23, 2021

Type of change

  • Bugfix
  • Feature
  • New bidder adapter
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Does this change affect user-facing APIs or examples documented on http://prebid.org?
  • Other

Description of change

  • test parameters for validating bids
{
  bidder: '<bidder name>',
  params: {
    // ...
  }
}

Be sure to test the integration with your adserver using the Hello World sample page.

  • contact email of the adapter’s maintainer
  • official adapter submission

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

Other information

@ChrisHuie
Copy link
Collaborator

@IOTiagoFaria it appears you are overwriting your entire current bid adapter file? I haven't compared the two files but is this intentional?

@ChrisHuie ChrisHuie self-requested a review April 24, 2021 10:46
@ChrisHuie ChrisHuie self-assigned this Apr 24, 2021
Co-authored-by: Surovenko Alexey <surovenko.alexey@gmail.com>
@IOTiagoFaria
Copy link
Contributor Author

@ChrisHuie The only changes where made to the "parseResponseOpenRTBToPrebidjs" function.
Not sure why it shows as a new  file

dreischer and others added 5 commits April 26, 2021 08:26
* RTD Provider rebase

* wrongly merged to master

* Added meta field for advertiser domains.
* Add files via upload

* Add files via upload

* Change params on gnetBidder

* ADJ - Use parseSizesInput to get sizes
ADJ - Check serverResponse object
ADJ - Remove getUserSyncs function

* ADJ - Change prebid endpoint

* ADJ - Change endpoint on test
* Added TheMediaGridNM Bid Adapter

* Updated required params for TheMediaGridNM Bid Adapter

* Update TheMediGridNM Bid Adapter

* Fix tests for TheMediaGridNM Bid Adapter

* Fixes after review for TheMediaGridNM Bid Adapter

* Add support of multi-format in TheMediaGrid Bid Adapter

* Update sync url for grid and gridNM Bid Adapters

* TheMediaGrid Bid Adapter: added keywords adUnit parameter

* Update TheMediaGrid Bid Adapter to support keywords from config

* Implement new request format for TheMediaGrid Bid Adapter

* Fix jwpseg params for TheMediaGrid Bid Adapter

* Update unit tests for The Media Grid Bid Adapter

* Fix typo in TheMediaGrid Bid Adapter

* Added test for jwTargeting in TheMediaGrid Bid Adapter

* The new request format was made by default in TheMediaGrid Bid Adapter

* Update userId format in ad request for TheMediaGrid Bid Adapter

* Added bidFloor parameter for TheMediaGrid Bid Adapter

* Fix for review TheMediaGrid Bid Adapter

* Support floorModule in TheMediaGrid Bid Adapter

* Fix empty bidfloor for TheMediaGrid Bid Adapter

* Some change to restart autotests

* Fix userIds format for TheMediaGrid Bid Adapter

* Remove digitrust userId from TheMediaGrid Bid Adapter

* Protocols was added in video section in ad request for TheMediaGrid Bid Adapter

* TheMediaGrid: fix trouble with alias using

* TheMediaGridNM: fix trouble with alias

* TheMediaGrid Bid Adapter: added support of PBAdSlot module

* TheMediaGrid Bid Adapter: fix typo
* Sonobi - Added Coppa Flag check

* added unit test for sonobi coppa flag
@patmmccann
Copy link
Collaborator

Includes change for #6466 thank you

patmmccann and others added 2 commits April 27, 2021 05:48
…dd support for meta.advertiserDomains (#6656)

* Update eids.js

* Update eids_spec.js

* Update eids.js

* Update pubmaticBidAdapter_spec.js

* Update eids.js

* Update eids_spec.js

* Update conversantBidAdapter_spec.js

* Update rubiconBidAdapter_spec.js

* Update conversantBidAdapter_spec.js

* Delete test/spec/adapters directory

* Update userId_spec.js

* Delete iasBidAdapter.js

* Add files via upload

* Update deepintentBidAdapter.js

* Update deepintentBidAdapter_spec.js

* Update rtbhouseBidAdapter.js

* Update rtbhouseBidAdapter_spec.js

* Update deepintentBidAdapter.js

* Update rtbhouseBidAdapter.js

* Update rtbhouseBidAdapter.js

* Update rtbhouseBidAdapter_spec.js

* Update deepintentBidAdapter.js

* Update rhythmoneBidAdapter.js

* Update rhythmoneBidAdapter_spec.js

* Update aolBidAdapter.js

* Update aolBidAdapter_spec.js

* Update aolBidAdapter.js

* Update aolBidAdapter.js

* Update teadsBidAdapter.js

* Update teadsBidAdapter_spec.js

* Update openxBidAdapter_spec.js

* Update openxBidAdapter.js

* Update openxBidAdapter.js

* Update openxBidAdapter_spec.js

* Update openxBidAdapter.js
@IOTiagoFaria
Copy link
Contributor Author

IOTiagoFaria commented Apr 27, 2021

@patmmccann are these the changes you requested? e1d7aa9

* RTD Provider rebase

* wrongly merged to master

* Added meta field for advertiser domains.

* Fixing bug where losing bids throw undefined into the response.

* Wrapped if-statement
@ChrisHuie
Copy link
Collaborator

@IOTiagoFaria can you please pull in recent merges so this pr doesn't view all these changes as new added files 🙏 Not sure what happened but hoping that fixes the git additions to sync

@patmmccann
Copy link
Collaborator

@patmmccann are these the changes you requested? e1d7aa9

Thanks for adding advertiserDomains, you have been removed from the list of bidders who do not support adomain.

@IOTiagoFaria
Copy link
Contributor Author

@ChrisHuie what is the status of this review?

@patmmccann
Copy link
Collaborator

@ChrisHuie what is the status of this review?

You have 37 files changed in this PR.

It's important to only submit your changes.

@IOTiagoFaria
Copy link
Contributor Author

My mistake, do you think it will be easier/faster if this PR got cancelled and I create a new one with the fork fully updated?

@patmmccann
Copy link
Collaborator

That's up to you; it's no trouble if you want to close and resubmit or prefer to fix this one

@IOTiagoFaria
Copy link
Contributor Author

I think its better for a new PR to be created, I don't want to mess this one even more...

@IOTiagoFaria
Copy link
Contributor Author

new PR created #6680

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.