-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Update Innity Adapter to Prebid.js v1.0 #2180
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@engtat Left some minor comments.
modules/innityBidAdapter.js
Outdated
method: 'GET', | ||
url: ENDPOINT, | ||
data: { | ||
cb: new Date().getTime(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is recommended to use utils.timestamp()
modules/innityBidAdapter.js
Outdated
}; | ||
return [bidResponse]; | ||
}, | ||
getUserSyncs: function(syncOptions, serverResponses) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can remove this function if you are not doing userSyncs.
Also please submit a PR to the docs repo to add a file for your adapter to the bidders directory so your adapter's params will appear on the bidders page. Thank you for contributing |
Changes updated. For PR to the doc repo, I actually posted this earlier in my PR (prebid/prebid.github.io#630) or I am missing something? Thanks. |
* master: Update Vertoz adapter for Prebid 1.0 (prebid#2104) Add multiple bids request (prebid#2136) [NEW Adapter] RTBHouseBidAdapter (prebid#2184) Update Innity Adapter to Prebid.js v1.0 (prebid#2180) Update Adyoulike Adapter to prebid 1.0 (prebid#2077) Change bidderCode for DAN Marketplace Bid Adapter (prebid#2183) only count bid timeouts if bidder didn't call done. fixes prebid#2146 (prebid#2154) [Edit BidAdapter] rxrtb adapter for Perbid.js 1.0 (prebid#2182) Update NasmediaAdmixer adapter (prebid#2164) only do video caching if we don't already have a videoCacheKey (prebid#2143)
* Innity Adapter for Prebid.js 1.0 * Fix Innity Adapter Test Spec Error (prebid#2180) * Fix utils.timestamp and remove userSyncs (prebid#2180)
* Innity Adapter for Prebid.js 1.0 * Fix Innity Adapter Test Spec Error (prebid#2180) * Fix utils.timestamp and remove userSyncs (prebid#2180) * Added Prebid.js 3.0 Support
* Innity Adapter for Prebid.js 1.0 * Fix Innity Adapter Test Spec Error (prebid#2180) * Fix utils.timestamp and remove userSyncs (prebid#2180) * Added Prebid.js 3.0 Support * Innity Bid Adapter: Add meta adomain support
Type of change
Description of change
Migrate to Prebid.js v1.0
Docs repo PR
prebid/prebid.github.io#630