Skip to content

Remove config setting from Optimatic adapter#1909

Merged
mkendall07 merged 2 commits intomasterfrom
optimatic-remove-config
Dec 6, 2017
Merged

Remove config setting from Optimatic adapter#1909
mkendall07 merged 2 commits intomasterfrom
optimatic-remove-config

Conversation

@jaiminpanchal27
Copy link
Collaborator

Type of change

  • Other

Description of change.

Setting config values in adapter is not allowed. This can be done via pbjs.setConfig method on page.
Removed config setting from Optimatic adapter

Other information

@optimatic58

@mkendall07
Copy link
Contributor

@jaiminpanchal27
2:10 error 'config' is defined but never used no-unused-vars

@optimatic58
Copy link
Contributor

Does this mean we have to ask pubs to set pbjs.setConfig({ usePrebidCache: true })? We are using vastXML in bidResponse and I understand that it will only work if the usePrebidCache is set to true. Please let us know. thanks!

@mkendall07
Copy link
Contributor

@optimatic58
Yes that's right you need to ask pubs to do pbjs.setConfig({ usePrebidCache: true }) in their setup.

@mkendall07 mkendall07 merged commit d79c1a2 into master Dec 6, 2017
@matthewlane matthewlane deleted the optimatic-remove-config branch December 6, 2017 21:07
Millerrok pushed a commit to Vertamedia/Prebid.js that referenced this pull request Dec 8, 2017
* 'master' of https://github.com/prebid/Prebid.js: (43 commits)
  Merge Prebid 1.0 to Master (prebid#1936)
  Prebid.js 0.34.1 release
  Vertamedia adapter outstream support (prebid#1860)
  Expose native image-type asset dimensions on bid response object (prebid#1919)
  Remove for of (prebid#1932)
  Unit-test fix (prebid#1927)
  Remove description_url (prebid#1922)
  Trion Interactive Adapter Bugfix (prebid#1925)
  Remove config setting from Optimatic adapter (prebid#1909)
  IE bug fix (prebid#1930)
  Clarify ad unit media filtering warning (prebid#1903)
  Add ReadPeak Bid Adapter (prebid#1838)
  Change clone function to make deep copies (prebid#1910)
  Serverbid 1.0 (prebid#1802)
  sekindoUM for prebid1.0 (prebid#1777)
  update auctionId to be requestId (prebid#1896)
  bug fixed to populate userSync default values (prebid#1897)
  Increment pre version
  AdkernelAdn analytics adapter (prebid#1868)
  Justpremium Adapter: use `filter` instead of `...new Set` (prebid#1895)
  ...
vzhukovsky added a commit to aol/Prebid.js that referenced this pull request Dec 28, 2017
….34.0 to aolgithub-master

* commit 'f0ba90afa8b52de7a646d43928b2d51ee42e74a1': (53 commits)
  Added changelog entry.
  Added partners ids.
  Added dynamic ttl property for One Display and One Mobile.
  Prebid.js 0.34.1 release
  Vertamedia adapter outstream support (prebid#1860)
  Expose native image-type asset dimensions on bid response object (prebid#1919)
  Remove for of (prebid#1932)
  Unit-test fix (prebid#1927)
  Remove description_url (prebid#1922)
  Trion Interactive Adapter Bugfix (prebid#1925)
  Remove config setting from Optimatic adapter (prebid#1909)
  IE bug fix (prebid#1930)
  Clarify ad unit media filtering warning (prebid#1903)
  Add ReadPeak Bid Adapter (prebid#1838)
  Change clone function to make deep copies (prebid#1910)
  Serverbid 1.0 (prebid#1802)
  sekindoUM for prebid1.0 (prebid#1777)
  update auctionId to be requestId (prebid#1896)
  bug fixed to populate userSync default values (prebid#1897)
  Increment pre version
  ...
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.

4 participants

Comments