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.

Sign up for GitHub

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

Jump to bottom

Apacdex Adapter: change endpoint url and TCF purpose 1 enforcement #8247

Merged
merged 1 commit into from
Apr 20, 2022
Merged

Apacdex Adapter: change endpoint url and TCF purpose 1 enforcement #8247

merged 1 commit into from
Apr 20, 2022

Conversation

thuyhq
Copy link
Contributor

@thuyhq thuyhq commented Apr 1, 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

  • Change adapter endpoint url
  • Only perform user sync when purpose 1 is passed

@ChrisHuie ChrisHuie self-requested a review April 6, 2022 16:23
@ChrisHuie ChrisHuie self-assigned this Apr 6, 2022
@patmmccann
Copy link
Collaborator

@thuyhq please document this and the use of gdpr enforcement module bidder override; a publisher likely doesnt want to do enforcement on your adapter bc you have conditional endpoints

@thuyhq
Copy link
Contributor Author

thuyhq commented Apr 8, 2022

@thuyhq please document this and the use of gdpr enforcement module bidder override; a publisher likely doesnt want to do enforcement on your adapter bc you have conditional endpoints

Hi @patmmccann,
I have limited knowledge of GDPR enforcement. Can you tell me what I need to do next?

@ChrisHuie ChrisHuie merged commit d1a0f09 into prebid:master Apr 20, 2022
JoelPM pushed a commit to JoelPM/Prebid.js that referenced this pull request Apr 25, 2022
@thuyhq @JoelPM
update apacdex bid adapter (prebid#8247)
f3792ff
@dgirardi dgirardi mentioned this pull request Apr 28, 2022
Merged
@saxsir saxsir mentioned this pull request Oct 5, 2022
Closed
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@patmmccann patmmccann patmmccann approved these changes

@ChrisHuie ChrisHuie Awaiting requested review from ChrisHuie

Assignees

@ChrisHuie ChrisHuie

Labels
maintenance needs docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@thuyhq @patmmccann @ChrisHuie