-
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
Ozone Bid Adapter: Various improvements #8755
Conversation
added instream support fixed/cleaned up code
Not sure why Circle CI tests aren't running - spec file looks okay to me - are you able to advise @patmmccann ? |
Typically if you're following your own fork it may not run |
I added some line breaks to your md file, that seemed to do the trick (bc I don't follow your fork in circleci) |
Thanks @patmmccann - good to know ! @ncolletti - let me know if you need anything to help with your review / approving this |
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.
lgtm
Looks good @AskRupert-DM -- Before I merge, could you open a PR with the prebid site repo updating your dev-doc to include instream and relevant bid params: https://github.com/prebid/prebid.github.io/blob/master/dev-docs/bidders/ozone.md |
done - thank you! |
Docs PR: |
* ozone adapter 2.8.0 added instream support fixed/cleaned up code * spec test for ozone 2.8.0 adapter * ozone adapter 2.8.0 * Update ozoneBidAdapter_spec.js * Update ozoneBidAdapter.md Co-authored-by: Patrick McCann <[email protected]>
* ozone adapter 2.8.0 added instream support fixed/cleaned up code * spec test for ozone 2.8.0 adapter * ozone adapter 2.8.0 * Update ozoneBidAdapter_spec.js * Update ozoneBidAdapter.md Co-authored-by: Patrick McCann <[email protected]>
Type of change
Description of change
Added Support for instream
Minor code clean ups
[email protected]