-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Risemediatech Bid Adapter : New Bidder Adapter #13600
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
Conversation
Risemediatech Bid Adapter : New Bidder Adapter
Tread carefully! This PR adds 2 linter errors (possibly disabled through directives):
|
We don't have a policy against brand confusion, but I'm curious, is this intentional |
Our legal entity is called Rise Media Technologies and there's no intentional connection or attempt to associate with the rise adapter. Our goal is to operate independently without creating confusion |
fair enough, i see there are at least a dozen companies called Rise in media or advertising and I am certainly not the police on that one |
Updated the documentation for the adapter and formatted adapter
@patmmccann Hope you are doing well. Can you please take a look at the adapter? |
@patmmccann Can you take a look |
You don't pass linting |
@patmmccann Resolved the same. Can you take a look? |
Type of change
Bugfix
Feature
[x ] 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
Added a bidder adapter for RisemediaTech
Maintainer : prebid@risemediatech.io
Sample Ad Units
Sample Ad Unit : Banner
Sample Ad Unit : Video
Bidder Params Documentation
prebid/prebid.github.io#6150