8000 GumGum: adds schain param by susyt · Pull Request #4442 · prebid/Prebid.js · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

GumGum: adds schain param #4442

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

Merged
merged 2 commits into from
Nov 21, 2019
Merged

GumGum: adds schain param #4442

merged 2 commits into from
Nov 21, 2019

Conversation

susyt
Copy link
Contributor
@susyt susyt commented Nov 8, 2019

Type of change

  • Feature

Description of change

Adds support for supply chain object using IAB's Non-OpenRTB serialization format

Copy link
Contributor
@sumit116 sumit116 left a comment

Choose a reason for hiding this comment

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

Hi @susyt, Can you add a docs PR to update the .md file of your adapter with schain_supported set to true and add the PR link here?

@susyt
Copy link
Contributor Author
susyt commented Nov 15, 2019

Hi @sumit116 , could you provide an example of where I should set schain_supported=true in the doc file? Should I add that to the description portion? I was reviewing other adapters who are now supporting schain and they didn't have any mentions of schain support in their md file.
thanks!

@sumit116
Copy link
Contributor

@susyt , you can refer to https://github.com/prebid/prebid.github.io/blob/master/dev-docs/bidders/pubmatic.md. This file contains what you need to add i.e schain_supported: true.

@susyt
Copy link
Contributor Author
susyt commented Nov 19, 2019

@sumit116 Thanks for the link! here's the other PR for updating the doc - prebid/prebid.github.io#1613
please let me know if there are any other changes needed

@susyt susyt requested a review from sumit116 November 20, 2019 19:27
@sumit116 sumit116 added LGTM and removed needs docs labels Nov 21, 2019
@sumit116 sumit116 merged commit 635c7cc into prebid:master Nov 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0