BidsCube Bid Adapter: add new adapter

[vlaktion]

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:

• A link to a PR on the docs repo at https://github.com/prebid/prebid.github.io/

Other information

[bretg]

docs PR prebid/prebid.github.io#2944

[vlaktion]

All done

[patmmccann]

You must return advertiserDomains

[patmmccann]

@vlaktion reminder that you must return advertiserDomains, see #6650 for reference

[msm0504]

@vlaktion The latest changes allowed me to get data from your server when using my test page. I've approved the PR.

However, before I merge it, please address @patmmccann's comment from last week.

[vlaktion]

@patmmccann I see the advertiserDomains is Optional. I don't have this information. Can you approve my PR?

[vlaktion]

@patmmccann All done! For test:
POST https://supply.bidscube.com/?c=o&m=multi
{
"bidId": "9ec5b177515ee2e5",
"bidder": "bidscube",
"placements": [{
"placementId": 0,
"traffic": "banner",
"allParams": "{}"
}]
}

[patmmccann]

Excellent! Can you add a unit test that demonstrates meta.advertiserDomains is set in the interpreted response?

[vlaktion]

@patmmccann Added a unit test that demonstrates meta.advertiserDomains is set in the interpreted response

[vlaktion]

Hi. Why is BidsCube adapter don`t exist in https://docs.prebid.org/download.html?

[vlaktion]

@patmmccann Hi. Can you answer on my question?