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

Brightroll auction type explicitly default to first price auction #746

Merged
merged 3 commits into from
Nov 13, 2018

Conversation

smithaammassamveettil
Copy link
Contributor

Description:

Explicity setting the auction type to 1 for first price auction in Brightroll adaptor

@smithaammassamveettil
Copy link
Contributor Author

smithaammassamveettil commented Nov 13, 2018

@dbemiller , Thanks for reviewing and approving. Can you please merge the changes to master as well?

@dbemiller
Copy link
Contributor

dbemiller commented Nov 13, 2018

Takes 2 approvals to merge. It will make it in eventually

Copy link
Collaborator

@hhhjort hhhjort left a comment

Choose a reason for hiding this comment

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

Looks good.

@dbemiller dbemiller merged commit 948ab28 into prebid:master Nov 13, 2018
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 1, 2020
…ebid#746)

* Brightroll auction type explicitly default to first price auction

* fixed merge issue

* removed unnessary code
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 2, 2020
…ebid#746)

* Brightroll auction type explicitly default to first price auction

* fixed merge issue

* removed unnessary code
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 4, 2020
…ebid#746)

* Brightroll auction type explicitly default to first price auction

* fixed merge issue

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

Successfully merging this pull request may close these issues.

3 participants