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

Update rubicon analytics adapter with wrapper name #3990

Merged

Conversation

idettman
Copy link
Contributor

@idettman idettman commented Jul 11, 2019

Type of change

  • Other

Description of change

Added wrapperName to the analytics event JSON. The wrapperName is retrieved from the config pathrubicon.wrapperName.

@idettman idettman requested a review from harpere July 11, 2019 20:36
@idettman idettman self-assigned this Jul 12, 2019
@idettman idettman requested a review from robertrmartinez July 16, 2019 23:02
@harpere harpere added LGTM and removed needs review labels Jul 23, 2019
Copy link
Collaborator

@robertrmartinez robertrmartinez left a comment

Choose a reason for hiding this comment

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

Cool

Have we confirmed with the R&I team that this is okay to be merged and the records will not fail?

@harpere harpere merged commit c2bdb94 into prebid:master Jul 23, 2019
leonardlabat pushed a commit to criteo-forks/Prebid.js that referenced this pull request Jul 30, 2019
* Add microadBidAdapter

* Remove unnecessary encodeURIComponent from microadBidAdapter

* Submit Advangelists Prebid Adapter

* Submit Advangelists Prebid Adapter 1.1

* Correct procudtion endpoint for prebid

* analytics update with wrapper name

* changed to only send wrapperName with event when it has a value

* changed wrapperName to use value from config 'rubicon.wrapperName'

* update tests for wrapperName

* update wrapperName as let for consistency

* updated wrapperName from let to const
VideoReach pushed a commit to VideoReach/Prebid.js that referenced this pull request Aug 1, 2019
* Add microadBidAdapter

* Remove unnecessary encodeURIComponent from microadBidAdapter

* Submit Advangelists Prebid Adapter

* Submit Advangelists Prebid Adapter 1.1

* Correct procudtion endpoint for prebid

* analytics update with wrapper name

* changed to only send wrapperName with event when it has a value

* changed wrapperName to use value from config 'rubicon.wrapperName'

* update tests for wrapperName

* update wrapperName as let for consistency

* updated wrapperName from let to const
@idettman idettman deleted the rubicon-analytics-adapter-add-wrappername branch September 10, 2019 16:48
@idettman idettman restored the rubicon-analytics-adapter-add-wrappername branch September 10, 2019 16:48
sa1omon pushed a commit to gamoshi/Prebid.js that referenced this pull request Nov 28, 2019
* Add microadBidAdapter

* Remove unnecessary encodeURIComponent from microadBidAdapter

* Submit Advangelists Prebid Adapter

* Submit Advangelists Prebid Adapter 1.1

* Correct procudtion endpoint for prebid

* analytics update with wrapper name

* changed to only send wrapperName with event when it has a value

* changed wrapperName to use value from config 'rubicon.wrapperName'

* update tests for wrapperName

* update wrapperName as let for consistency

* updated wrapperName from let to const
@idettman idettman deleted the rubicon-analytics-adapter-add-wrappername branch February 29, 2020 06:14
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.

3 participants