-
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
Single Adform adapter #6840
Single Adform adapter #6840
Conversation
(cherry picked from commit 6580bf4)
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.
I think this PR should only remove adform bid adapter? You shouldn't be adding changes to the adfBidAdapter
since those changes are already merged in #6726.
Should I revert commit 0ada455 ? Changes would be applied by some later merge from master? |
Yes, as I see you only need 1st commit, which removes |
…rebid#6726)" This reverts commit 0ada455.
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
did your fix for #6650 geet removed from this pr? |
as @FilipStamenkovic noticed - it's ok to have it only in master branch. Later changes will be merged. |
confirming i see it on master, ty |
Type of change
Description of change
Removed legacy Adform adapter, since prebid 5.0 only adf adapter should be used to fetch Adform demand.
contact email of the adapter’s maintainer [email protected]
official adapter submission
A link to a PR on the docs repo at https://github.com/prebid/prebid.github.io/
Single adform bid adapter prebid.github.io#2991
Other information
Fixes #6650