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

Tappx Bid Adapter: accept ortb2 country #9907

Merged
merged 23 commits into from
Jul 26, 2023
Merged

Conversation

prebidtappx
Copy link
Contributor

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

Changes related to issue #9676 accepting Geo data (more precisely Country) from ORTB2 first-party data.

Other information

Related issue #9676

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.

Also needs tests,
Thanks!

@patmmccann patmmccann self-assigned this May 6, 2023
@patmmccann patmmccann changed the title Tappx: Adapters that accept geolocation data from bid parameters should also accept it from ortb2 Tappx Bid Adapter: accept ortb2 country May 6, 2023
please add unit tests
@patmmccann
Copy link
Collaborator

@prebidtappx please add unit tests here to make sure we haven't broken you.

@patmmccann patmmccann removed their assignment Jul 19, 2023
@patmmccann patmmccann merged commit 9345b0a into prebid:master Jul 26, 2023
santii7395 pushed a commit to themaven-net/Prebid.js that referenced this pull request Aug 28, 2023
* Fix: creating host correctly when http or https are added from the beginning

* Fix :: Changed double quotes for single quotes

* Fix :: Getting the full page URL

* Fix :: Changed order params

* Fix :: Replaced quotes from double to simple

* Fix :: Adapting format to lint

* Remove TODO comment

* Added more controls

* camelcase fix

* Changed test

* Remove "inIframe" util

* Fix: solved error getOs and created getVendor functions

* Fix: Solved CircelCI format problems

* 10207_include setConfig function and intruccions in readme file

* Update tappxBidAdapter.md

* Update tappxBidAdapter.js

* Fix: repriorising to get the OS correctly

* Fix: Correcting EsLint requirements for merge

* Update tappxBidAdapter.js

please add unit tests

---------

Co-authored-by: Jordi Arnau <[email protected]>
Co-authored-by: ruben_tappx <[email protected]>
Co-authored-by: jgarciaorad <[email protected]>
Co-authored-by: Patrick McCann <[email protected]>
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