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

PubmaticBidAdapter: Using plcmt instead of placement #11623

Merged
merged 22 commits into from
May 30, 2024

Conversation

pm-nitin-shirsat
Copy link
Contributor

Type of change

  • Bugfix

  • Feature

  • New bidder adapter

  • Updated bidder adapter

  • Code style update (formatting, local variables)

  • Refactoring (no functional changes, no api changes)

  • Build related changes

  • CI related changes

  • Does this change affect user-facing APIs or examples documented on http://prebid.org?

  • Other

Description of change

PubmaticBidAdapter: Using plcmt instead of placement

Other information

pm-nitin-shirsat and others added 22 commits October 11, 2022 13:57
Sync up prebid master to fork master
…bid-master

# Conflicts:
#	modules/pubmaticBidAdapter.js
#	test/spec/modules/pubmaticBidAdapter_spec.js
Sync with prebid master
Merge pull request #5 from pm-nitin-shirsat/master
Sync with prebid master
Sync with Prebidd.js master
Sync with prebid master
@patmmccann patmmccann self-assigned this May 30, 2024
@ChrisHuie ChrisHuie merged commit 7defcab into prebid:master May 30, 2024
4 checks passed
DecayConstant pushed a commit to mediavine/Prebid.js that referenced this pull request Jul 18, 2024
* Implement functionality for deal priority

* Update test cases

* kick off test manually

* Added support of GPP to PubMatic adapter

* gpp_sid in user syncs supposed to encode as a string, not an array

* Remove extra space

* Remove trailing spaces

* Remove the placement parameter and update test cases accordingly, Add plcmt parameter.

* Supporting placement parameter and logging warning message, for the plcmt parameter, if it is missing.

* Remove commented code

---------

Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com>
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.

3 participants