Skip to content
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

Merged
merged 3 commits into from
May 31, 2021
Merged

Conversation

braizhas
Copy link
Contributor

Type of change

  • Other

Description of change

Removed legacy Adform adapter, since prebid 5.0 only adf adapter should be used to fetch Adform demand.

Other information

Fixes #6650

Copy link
Contributor

@FilipStamenkovic FilipStamenkovic left a 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.

@braizhas
Copy link
Contributor Author

Should I revert commit 0ada455 ? Changes would be applied by some later merge from master?

@FilipStamenkovic
Copy link
Contributor

Yes, as I see you only need 1st commit, which removes adform bidder and adds alias in adfBidAdapter, all your other changes are already on master branch.

Copy link
Contributor

@FilipStamenkovic FilipStamenkovic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@FilipStamenkovic FilipStamenkovic merged commit 1001c00 into prebid:prebid-5.0 May 31, 2021
@patmmccann
Copy link
Collaborator

did your fix for #6650 geet removed from this pr?

@braizhas
Copy link
Contributor Author

braizhas commented Jun 1, 2021

as @FilipStamenkovic noticed - it's ok to have it only in master branch. Later changes will be merged.

@patmmccann
Copy link
Collaborator

confirming i see it on master, ty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants