Skip to content

Add some missing deprecations for 1.0#383

Merged
mkendall07 merged 2 commits intomasterfrom
update_deprecations
Sep 28, 2017
Merged

Add some missing deprecations for 1.0#383
mkendall07 merged 2 commits intomasterfrom
update_deprecations

Conversation

@mkendall07
Copy link
Contributor

- `pbjs.allBidsAvailable`
- `pbjs.buildMasterVideoTagFromAdserverTag` in favor of `pbjs.adServers.dfp.buildVideoUrl`
- `adUnit.sizeMapping` - see below for [details on sizeConfig]({{site.baseurl}}/dev-docs/prebid-1.0-API.html#size-mapping-changes)

Copy link
Collaborator

Choose a reason for hiding this comment

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

$$PREBID_GLOBAL$$.bidderTimeout - Moved to config

prebid/Prebid.js#1633 PR is not merged but it removes
$$PREBID_GLOBAL$$._adsReceived (You already added that)
$$PREBID_GLOBAL$$.timeoutBuffer - Moved to config
$$PREBID_GLOBAL$$.logging - Moved to config
$$PREBID_GLOBAL$$.publisherDomain - Moved to config
$$PREBID_GLOBAL$$.setBidderSequence - Moved to config
$$PREBID_GLOBAL$$.setS2SConfig - Moved to config

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jaiminpanchal27 what is $$PREBID_GLOBAL$$.bidderTimeout used for? Shouldn't this just be set in requestBids?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added pbjs.timeoutBuffer but the rest are covered in the pbjs.setConfig section below this.

Copy link
Collaborator

@jaiminpanchal27 jaiminpanchal27 left a comment

Choose a reason for hiding this comment

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

Some more to be added to list. Left a comment

@mkendall07 mkendall07 merged commit 538f4d8 into master Sep 28, 2017
@mkendall07 mkendall07 deleted the update_deprecations branch September 28, 2017 22:13
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.

3 participants

Comments