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

PubMatic to support passing content object set in pbjs.setConfig #5592

Merged
merged 57 commits into from
Aug 14, 2020

Conversation

pm-harshad-mane
Copy link
Contributor

Type of change

  • Feature

Description of change

PubMatic to support passing content object set in pbjs.setConfig

pbjs.setConfig("content", {})

@pm-harshad-mane
Copy link
Contributor Author

Hello @jsnellbaker
Could you please review this PR?

@pm-harshad-mane
Copy link
Contributor Author

Hello @robertrmartinez ,
Could you please review this PR?

@jsnellbaker jsnellbaker merged commit 7ede93b into prebid:master Aug 14, 2020
BrightMountainMediaInc pushed a commit to BrightMountainMediaInc/Prebid.js that referenced this pull request Sep 14, 2020
…bid#5592)

* added support for pubcommon, digitrust, id5id

* added support for IdentityLink

* changed the source for id5

* added unit test cases

* changed source param for identityLink

* read content object from config and send it in site.content and app.content

* do not use content object from config if content object is present in app as app.content

* fixed the test-cases

* app.content related test cases
BrightMountainMediaInc pushed a commit to BrightMountainMediaInc/Prebid.js that referenced this pull request Sep 14, 2020
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.

2 participants