-
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
Lifestreet Bid adapter: initial release #8340
Conversation
please note there are no tests in your file |
oh sorry! can't get how could this happens.. added back |
@ChrisHuie could you please tell me which kind of update needs to be done? Is it something from my side or from your side? Thanks! |
Yes. My apologies. Your tests seem to be failing their unit tests per CircleCi. |
@ChrisHuie sorry for radio silence! I've fixed tests, could you please take a look on this PR? |
@ChrisHuie additional ping just in case you you missed previous one |
Sorry I did. Will work on this review :) |
any news on this? |
*/ | ||
function formatBidRequest(bid, bidderRequest = {}) { | ||
const {params} = bid; | ||
const {referer} = (bidderRequest.refererInfo || {}); |
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.
FYI. refererInfo
was updated/refactored in Prebid 7 with #8450 . Not sure if you are expecting refererInfo to act as previous versions.
@probably-kira hoping to get this merged this week or next week. I don't see docs in our docs repo though (https://github.com/prebid/prebid.github.io). If you don't mind submitting a pr over there also to compliment this new adapter code pr. Here is some docs around how the markdown can be easily setup (https://docs.prebid.org/dev-docs/bidder-adaptor.html#submitting-your-adapter) :) |
hi Chris, I apologize, was on vacation. Will do everything till EOW |
@ChrisHuie I've checked and here is the file: |
thanks a lot @ChrisHuie!! |
* Lifestreet adapter with meta support added * Added tests * fix linting error * Fixed typo in tests * CR fix: import functions from utils instead of using * Co-authored-by: Chris Huie <[email protected]>
* Lifestreet adapter with meta support added * Added tests * fix linting error * Fixed typo in tests * CR fix: import functions from utils instead of using * Co-authored-by: Chris Huie <[email protected]>
Type of change
Description of change
Added back Lifestreet adapter with
meta
properties addedBe sure to test the integration with your adserver using the Hello World sample page.
For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:
Our docs exists: https://github.com/prebid/prebid.github.io/blob/master/dev-docs/bidders/lifestreet.md
Other information