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

Triplelift Bid Adapter: get opeCloud data from local storage #8542

Merged
merged 49 commits into from
Jun 8, 2022

Conversation

nllerandi3lift
Copy link
Contributor

Type of change

  • Feature

Description of change

Allows for our bid adapter to pick up the opeCloud localStorage (a Triplelift technology) and add to the bid request. This localStorage is set independently of Prebid.

Includes 3 tests:

  • should send 1PlusX data as fpd if localStorage is available and no other fpd is defined
  • should append 1PlusX data to existing user.data entries if localStorage is available
  • should not append anything if getDataFromLocalStorage returns null

To Do

  • permit bidder to access storage

Dan Goldin and others added 30 commits January 5, 2022 23:07
Catch TL Prebid up to the upstream
…e-update

TL-19850 Finished log error logic around floors functionality
@nllerandi3lift nllerandi3lift marked this pull request as ready for review June 8, 2022 15:47
Copy link
Collaborator

@patmmccann patmmccann left a comment

Choose a reason for hiding this comment

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

Minor tweak, ty!

@patmmccann patmmccann merged commit 9f14ab2 into prebid:master Jun 8, 2022
@patmmccann
Copy link
Collaborator

@nllerandi3lift feel free to resubmit on 6.29.x-legacy branch if this is important to your company; we'll see legacy releases continue for a few weeks

renebaudisch pushed a commit to renebaudisch/Prebid.js that referenced this pull request Jun 28, 2022
…8542)

* TL-19850 Finished log error logic around floors functionality

* deprecates getlegacyFpd

* remove console log

* TL-19850 Changed functionlity of tests

* restore logErrorSpy aftereach

* passes 1px segments

* array and malformed json check

* spell check

* single quotes

* TL-27574 Second draft of implementing 1plusx logic

* cleanup around fetchOnePlusX

* replaces segment for ext

* ext should be object

* tests

* update localstorage key name

* rename to opecloud

* add frametype to tlBidAdapter

* removes comments

* clean up + no frame ft

* addressing feedback

* Revert "TL-27859: add frame type to prebid adapter"

* cleanup

* updates tests to work with prebid 7

* permission to access storage

* reset prebid global

Co-authored-by: Dan Goldin <[email protected]>
Co-authored-by: Patrick Loughrey <[email protected]>
Co-authored-by: Patrick Loughrey <[email protected]>
Co-authored-by: gbalboa0 <[email protected]>
Co-authored-by: gbalboa0 <[email protected]>
bwhisp pushed a commit to bwhisp/Prebid.js that referenced this pull request Jul 13, 2022
…8542)

* TL-19850 Finished log error logic around floors functionality

* deprecates getlegacyFpd

* remove console log

* TL-19850 Changed functionlity of tests

* restore logErrorSpy aftereach

* passes 1px segments

* array and malformed json check

* spell check

* single quotes

* TL-27574 Second draft of implementing 1plusx logic

* cleanup around fetchOnePlusX

* replaces segment for ext

* ext should be object

* tests

* update localstorage key name

* rename to opecloud

* add frametype to tlBidAdapter

* removes comments

* clean up + no frame ft

* addressing feedback

* Revert "TL-27859: add frame type to prebid adapter"

* cleanup

* updates tests to work with prebid 7

* permission to access storage

* reset prebid global

Co-authored-by: Dan Goldin <[email protected]>
Co-authored-by: Patrick Loughrey <[email protected]>
Co-authored-by: Patrick Loughrey <[email protected]>
Co-authored-by: gbalboa0 <[email protected]>
Co-authored-by: gbalboa0 <[email protected]>
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