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

[33Across adapter] Map ad unit path to element id #3920

Merged
merged 59 commits into from
Jun 25, 2019

Conversation

glebglushtsov
Copy link
Contributor

@glebglushtsov glebglushtsov commented Jun 18, 2019

Type of change

  • Bugfix
  • Feature
  • New 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

Use GPT capabilities to map ad unit path to element id when ad slots were configured with ad unit paths instead of element ids.

curlyblueeagle and others added 30 commits August 3, 2018 12:36
# Conflicts:
#	test/spec/modules/33acrossBidAdapter_spec.js
add withSite() method to TtxRequestBuilder
contribute viewability as 0 for inactive tab
Copy link
Contributor

@mike-chowla mike-chowla left a comment

Choose a reason for hiding this comment

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

LGTM

@mike-chowla mike-chowla merged commit b6aaf64 into prebid:master Jun 25, 2019
VideoReach pushed a commit to VideoReach/Prebid.js that referenced this pull request Aug 1, 2019
* fix return of _mapAdUnitPathToElementId()

* improve logging of _mapAdUnitPathToElementId()

* do not use Array.find()

* return id once element is found

* return id once element is found

* let -> const
sa1omon pushed a commit to gamoshi/Prebid.js that referenced this pull request Nov 28, 2019
* fix return of _mapAdUnitPathToElementId()

* improve logging of _mapAdUnitPathToElementId()

* do not use Array.find()

* return id once element is found

* return id once element is found

* let -> const
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.

4 participants