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

1plusX Rtd Module: Add FPID support for PAPI and Fix Test Bug #9115

Merged
merged 1 commit into from
Oct 17, 2022

Conversation

luca1x
Copy link
Contributor

@luca1x luca1x commented Oct 14, 2022

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

  • Extract 1plusX first party id and attach it to our profile API call in case it is present
  • Slightly relax the conditions of the gdpr consent object -- the consent string doesn't have to be defined anymore (e.g. if gdpr does not apply)
  • Fix a bug in a previous test case that made the test go green no matter what

Other information

@luca1x luca1x changed the title [DRAFT] 1plusX Adapter: Add FPID support for PAPI and Fix Test Bug 1plusX Adapter: Add FPID support for PAPI and Fix Test Bug Oct 17, 2022
@luca1x luca1x changed the title 1plusX Adapter: Add FPID support for PAPI and Fix Test Bug 1plusX Rtd Module: Add FPID support for PAPI and Fix Test Bug Oct 17, 2022
@ChrisHuie ChrisHuie self-requested a review October 17, 2022 19:42
@ChrisHuie ChrisHuie self-assigned this Oct 17, 2022
@ChrisHuie ChrisHuie merged commit 86db8eb into prebid:master Oct 17, 2022
JacobKlein26 pushed a commit to nextmillenniummedia/Prebid.js that referenced this pull request Feb 9, 2023
jorgeluisrocha pushed a commit to jwplayer/Prebid.js that referenced this pull request May 23, 2023
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.

2 participants