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

Add support for Gamoshi's Gambid Bid Adapter #2625

Merged
merged 8 commits into from May 30, 2018

Conversation

arikkfir
Copy link
Contributor

@arikkfir arikkfir commented May 25, 2018

Type of change

  • New bidder adapter

Description of change

Add support for Gamoshi's Gambid Platform to Prebid.js in the form of a new bidder adapter.
Here's how to simulate an example bid for testing:

{
  bidder: 'gambid',
  params: {
    supplyPartnerId: "1157"
  }
}

Contact email of the adapter’s maintainer: arik@gamoshi.com

@mkendall07
Copy link
Member

This pull request introduces 1 alert when merging 9a71939 into f7cf6cb - view on lgtm.com

new alerts:

  • 1 for Useless conditional

Comment posted by lgtm.com

@mkendall07
Copy link
Member

This pull request introduces 1 alert when merging f7bf27d into f7cf6cb - view on lgtm.com

new alerts:

  • 1 for Useless conditional

Comment posted by lgtm.com

@arikkfir arikkfir changed the title Initial submission of Gamoshi's Gambid Bid Adapter Add support for Gamoshi's Gambid Bid Adapter May 25, 2018
@ndhimehta ndhimehta assigned snapwich and unassigned ndhimehta May 29, 2018
@snapwich snapwich self-requested a review May 29, 2018 20:07
Copy link
Collaborator

@snapwich snapwich left a comment

Choose a reason for hiding this comment

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

It looks like you're missing the new GDPR compliance details. looks good other than that. http://prebid.org/dev-docs/modules/consentManagement.html

This commit utilizes Prebid.js's GDPR framework for passing user's
consent to our server. This includes both the actual bid request & the
user-syncs request.
@arikkfir
Copy link
Contributor Author

Thanks @snapwich - done.

@snapwich snapwich merged commit 3ee3b16 into prebid:master May 30, 2018
@arikkfir
Copy link
Contributor Author

@snapwich quick question - so on the next Prebid.js release users will be able to check the "Gamoshi Gambid" check-box?
Is there anything else we should do? do we need to create a PR for the docs repo as well?

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.

None yet

4 participants