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

LuponMedia: add bidfloor support #7458

Closed
wants to merge 12 commits into from
Closed

LuponMedia: add bidfloor support #7458

wants to merge 12 commits into from

Conversation

adxpremium
Copy link
Contributor

Type of change

  • Bugfix
  • [ x] 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?
  • [x ] Other

Description of change

Prebid-js v5.X support

}
bidFloor = typeof floorInfo === 'object' && floorInfo.currency === 'USD' && !isNaN(parseInt(floorInfo.floor)) ? parseFloat(floorInfo.floor) : undefined;
} else {
bidFloor = parseFloat(utils.deepAccess(bidRequest, 'params.floor'));
Copy link
Collaborator

@ChrisHuie ChrisHuie Oct 1, 2021

Choose a reason for hiding this comment

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

We have made it so that all adapters now only import the needed "utils" functions so you no longer need utils. in the root of the function call anymore. That is the code conflict (at least in the main code, not sure what the conflict in the testing file is).

@ChrisHuie ChrisHuie changed the title LuponMedia - v5.X support LuponMedia: add bidfloor and adomain support Oct 1, 2021
@ChrisHuie ChrisHuie changed the title LuponMedia: add bidfloor and adomain support LuponMedia: add bidfloor support Oct 1, 2021
@ChrisHuie
Copy link
Collaborator

@adxpremium I am not sure why the testing all the sudden stopped running on this pr?? Looks like both CircleCI and LGTM did noot run for the last 2 commits

@adxpremium
Copy link
Contributor Author

@ChrisHuie we have fixed the utils part. Checks are failing again. Could you please check what needs to be done? Thanks

@ChrisHuie
Copy link
Collaborator

@adxpremium your current branch is really far behind master. Pulling all prs from master should fix this because this adapter was removed in 5.0 (current version). This should add the whole adapter back into version 5.

ChrisHuie added a commit to prebid/prebid.github.io that referenced this pull request Oct 14, 2021
@adxpremium adxpremium closed this Oct 19, 2021
fowler446 pushed a commit to prebid/prebid.github.io that referenced this pull request Nov 19, 2021
* Lupon Media Bid Adapter: update prebid 5 complianc

per pr -> prebid/Prebid.js#7458

* update not in 5.x
bretg added a commit to prebid/prebid.github.io that referenced this pull request Dec 6, 2021
* Create generic bidder documentation (#3413)

* Fixed typo

* Added Generic bidder docs

* iProm Bid Adapter: update versions included (#3420)

per pr -> prebid/Prebid.js#7700

* add first-party data support to beachfront doc (#3418)

Co-authored-by: John Salis <john@beachfront.com>

* new bidderSettings field - allowZeroCpmBids (#3414)

* add native support (#3409)

* Missena Bid Adapter: update with prebid compliance (#3408)

* Missena Bid Adapter: update with prebid compliance

per pr -> prebid/Prebid.js#7648

* update versioning to include not in 5.x

* Add Intersection RTD Provider docs (#3407)

* upd docs

* Update dev-docs/modules/intersectionRtdProvider.md

Co-authored-by: Andrew <fowler446@users.noreply.github.com>

* Limelight Digital Bid Adapter: Add optional Publisher ID field (#3405)

* smartx Bid Adapter: refactor descriptions and adding "visibilityThreshold" (#3403)

* initial commit

* adjustments

* adjustments

* adjustments

* typo, further adjustments

* removed userIds and schain support

* Added optionals to outstream and made formatting

* Update smartx.md

minor edits for clarification and readability.

* adjusted outstream_options

* remove outstream_function, added pbjs_version_notes

* bugfix outstream options for default outstream renderer configuration

* refactor descriptions and adding "visibilityThreshold"

Co-authored-by: Gino <adtech@smartclip.tv>
Co-authored-by: Jean Stemp <38964447+jeanstemp@users.noreply.github.com>

* VIS.X: update docs (#3402)

Update the VIS.X documentation under the latest adapter changes.

* Adplus bid adapter doc added (#3389)

* Adplus bid adapter doc added

* Quotes added to string arguments

Co-authored-by: TCCDENIZ <cansin.deniz@turkcell.com.tr>

* Created gb_xandr.md docs (#3384)

* Create gb_xandr.md

* renamed the file

* Update dev-docs/bidders/goldbach.md

Co-authored-by: Andrew <fowler446@users.noreply.github.com>

* Lupon Media Bid Adapter: update prebid 5 complianc (#3347)

* Lupon Media Bid Adapter: update prebid 5 complianc

per pr -> prebid/Prebid.js#7458

* update not in 5.x

* Update userId.md (#2788)

* gpid docs (#3421)

* fixed broken link (#3423)

* fixed floors doc issue (#3425)

* added api version note (#3434)

* Interstitials (#2869)

* removed sizes from RP bid params

* added interstitial page and config to adUnit

* interstitial page corrections

* Update InterstitialAds.md

* Update InterstitialAds.md

* modified sidebar link

* changed interstitials file name and reference

* wordsmithing

Co-authored-by: MartianTribe <steve@martiantribe.com>
Co-authored-by: bretg <bgorsline@gmail.com>

* added vendor-specific flag (#3437)

* added KVP reference

Co-authored-by: Alex Maltsev <and1sscsgo@gmail.com>
Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com>
Co-authored-by: John Salis <jsalis531@gmail.com>
Co-authored-by: John Salis <john@beachfront.com>
Co-authored-by: jsnellbaker <31102355+jsnellbaker@users.noreply.github.com>
Co-authored-by: AdmixerTech <35560933+AdmixerTech@users.noreply.github.com>
Co-authored-by: Andrew <fowler446@users.noreply.github.com>
Co-authored-by: Ilya Medvedev <medved55rus@gmail.com>
Co-authored-by: Skylinar <53079123+Skylinar@users.noreply.github.com>
Co-authored-by: Gino <adtech@smartclip.tv>
Co-authored-by: Jean Stemp <38964447+jeanstemp@users.noreply.github.com>
Co-authored-by: rrelict <rrelict@users.noreply.github.com>
Co-authored-by: cnsndnz <82813495+cnsndnz@users.noreply.github.com>
Co-authored-by: TCCDENIZ <cansin.deniz@turkcell.com.tr>
Co-authored-by: dveljovicTX <69788339+dveljovicTX@users.noreply.github.com>
Co-authored-by: Patrick McCann <patmmccann@gmail.com>
Co-authored-by: Bryan Szekely <49168870+bszekely1@users.noreply.github.com>
Co-authored-by: MartianTribe <steve@martiantribe.com>
bretg added a commit to prebid/prebid.github.io that referenced this pull request Dec 8, 2021
* Create generic bidder documentation (#3413)

* Fixed typo

* Added Generic bidder docs

* iProm Bid Adapter: update versions included (#3420)

per pr -> prebid/Prebid.js#7700

* add first-party data support to beachfront doc (#3418)

Co-authored-by: John Salis <john@beachfront.com>

* new bidderSettings field - allowZeroCpmBids (#3414)

* add native support (#3409)

* Missena Bid Adapter: update with prebid compliance (#3408)

* Missena Bid Adapter: update with prebid compliance

per pr -> prebid/Prebid.js#7648

* update versioning to include not in 5.x

* Add Intersection RTD Provider docs (#3407)

* upd docs

* Update dev-docs/modules/intersectionRtdProvider.md

Co-authored-by: Andrew <fowler446@users.noreply.github.com>

* Limelight Digital Bid Adapter: Add optional Publisher ID field (#3405)

* smartx Bid Adapter: refactor descriptions and adding "visibilityThreshold" (#3403)

* initial commit

* adjustments

* adjustments

* adjustments

* typo, further adjustments

* removed userIds and schain support

* Added optionals to outstream and made formatting

* Update smartx.md

minor edits for clarification and readability.

* adjusted outstream_options

* remove outstream_function, added pbjs_version_notes

* bugfix outstream options for default outstream renderer configuration

* refactor descriptions and adding "visibilityThreshold"

Co-authored-by: Gino <adtech@smartclip.tv>
Co-authored-by: Jean Stemp <38964447+jeanstemp@users.noreply.github.com>

* VIS.X: update docs (#3402)

Update the VIS.X documentation under the latest adapter changes.

* Adplus bid adapter doc added (#3389)

* Adplus bid adapter doc added

* Quotes added to string arguments

Co-authored-by: TCCDENIZ <cansin.deniz@turkcell.com.tr>

* Created gb_xandr.md docs (#3384)

* Create gb_xandr.md

* renamed the file

* Update dev-docs/bidders/goldbach.md

Co-authored-by: Andrew <fowler446@users.noreply.github.com>

* Lupon Media Bid Adapter: update prebid 5 complianc (#3347)

* Lupon Media Bid Adapter: update prebid 5 complianc

per pr -> prebid/Prebid.js#7458

* update not in 5.x

* Update userId.md (#2788)

* gpid docs (#3421)

* fixed broken link (#3423)

* fixed floors doc issue (#3425)

* added api version note (#3434)

* Interstitials (#2869)

* removed sizes from RP bid params

* added interstitial page and config to adUnit

* interstitial page corrections

* Update InterstitialAds.md

* Update InterstitialAds.md

* modified sidebar link

* changed interstitials file name and reference

* wordsmithing

Co-authored-by: MartianTribe <steve@martiantribe.com>
Co-authored-by: bretg <bgorsline@gmail.com>

* added vendor-specific flag (#3437)

* added KVP reference

* review comments

* adding link to related reading

Co-authored-by: Alex Maltsev <and1sscsgo@gmail.com>
Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com>
Co-authored-by: John Salis <jsalis531@gmail.com>
Co-authored-by: John Salis <john@beachfront.com>
Co-authored-by: jsnellbaker <31102355+jsnellbaker@users.noreply.github.com>
Co-authored-by: AdmixerTech <35560933+AdmixerTech@users.noreply.github.com>
Co-authored-by: Andrew <fowler446@users.noreply.github.com>
Co-authored-by: Ilya Medvedev <medved55rus@gmail.com>
Co-authored-by: Skylinar <53079123+Skylinar@users.noreply.github.com>
Co-authored-by: Gino <adtech@smartclip.tv>
Co-authored-by: Jean Stemp <38964447+jeanstemp@users.noreply.github.com>
Co-authored-by: rrelict <rrelict@users.noreply.github.com>
Co-authored-by: cnsndnz <82813495+cnsndnz@users.noreply.github.com>
Co-authored-by: TCCDENIZ <cansin.deniz@turkcell.com.tr>
Co-authored-by: dveljovicTX <69788339+dveljovicTX@users.noreply.github.com>
Co-authored-by: Patrick McCann <patmmccann@gmail.com>
Co-authored-by: Bryan Szekely <49168870+bszekely1@users.noreply.github.com>
Co-authored-by: MartianTribe <steve@martiantribe.com>
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.

3 participants