Skip to content

Resetdigital bid Adapter: add fields and suport for coppa #9400

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

Merged
merged 9 commits into from
Feb 24, 2023
13 changes: 13 additions & 0 deletions modules/resetdigitalBidAdapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,16 +43,29 @@ export const spec = {
};
}

if (bidderRequest && bidderRequest.uspConsent) {
payload.ccpa = bidderRequest.uspConsent;
}

for (let x = 0; x < validBidRequests.length; x++) {
let req = validBidRequests[x]

payload.imps.push({
pub_id: req.params.pubId,
site_id: req.params.siteID ? req.params.siteID : null,
placement_id: req.params.placement ? req.params.placement : null,
position: req.params.position ? req.params.position : null,
bid_floor: req.params.bidFloor ? req.params.bidFloor : null,
lat_long: req.params.latLong ? req.params.latLong : null,
inventory: req.params.inventory ? req.params.inventory : null,
visitor: req.params.visitor ? req.params.visitor : null,
keywords: req.params.keywords ? req.params.keywords : null,
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is also true about keywords, the bid request object should have keywords you can append to the keywords you see in params

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I reviewed other adapters and couldn't find one that takes the keywords from the bid request object, so I think we are covered about that. in another case can you share an example with me?

Copy link
Collaborator

Choose a reason for hiding this comment

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

See issue #8660

zone_id: req.params.zoneId,
bid_id: req.bidId,
imp_id: req.transactionId,
sizes: req.sizes,
force_bid: req.params.forceBid,
coppa: req.ortb2.regs ? req.ortb2.regs.coppa : 0,
media_types: deepAccess(req, 'mediaTypes')
});
}
Expand Down
28 changes: 27 additions & 1 deletion modules/resetdigitalBidAdapter.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,17 +21,43 @@ Video is supported but requires a publisher supplied renderer at this time.
mediaTypes: {
banner: {
sizes: [[300,250]]
},

},
bids: [
{
bidder: "resetdigital",
params: {
pubId: "your-pub-id",
site_id: "your-site-id",
forceBid: true,
}
}
]
}
];


var videoAdUnits = [
{
code: 'your-div',
mediaTypes: {
video: {
playerSize: [640, 480]
},

},
bids: [
{
bidder: "resetdigital",
params: {
pubId: "your-pub-id",
forceBid: true
site_id: "your-site-id",
forceBid: true,
}
}
]
}
];

```