Skip to content

Feature/adshield - #99

Open
dpiwko19 wants to merge 3 commits into
dreamlab-masterfrom
feature/adshield
Open

dpiwko19 wants to merge 3 commits into
dreamlab-masterfrom
feature/adshield

Conversation

@dpiwko19

Copy link
Copy Markdown

Type of change

  • Bugfix

  • Feature

  • New bidder adapter

  • Updated 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

Other information

Comment thread modules/dasBidAdapter.js
}

if (bidderRequest.device) {
request.device = bidderRequest.device;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pasowałoby to przekształcić na int, bo DAS Bidder ma ścisłe typowanie i wygląda na to, że jakby ktoś tutaj przesłał string, to całe zapytanie będzie odrzucone.

Comment thread modules/dasBidAdapter.js Outdated
// Adblock detected on the page (set by dlApi as customParams.adf, the same `adf` CSR
// gets). Plain ext field so das-bidder can apply the creatives' adblock compatibility.
if (customParams.adf) {
request.ext.adf = customParams.adf;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants