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

Check global and custom vendor for Prebid #1261

Merged
merged 6 commits into from
Feb 12, 2024
Merged

Conversation

arelra
Copy link
Member

@arelra arelra commented Feb 9, 2024

What does this change?

The TCF global Prebid vendor has been dropped from Sourcepoint which means Prebid header bidding is failing for newly consented users in TCF.

This PR adds an additional check for consent for the new custom Prebid vendor so that we maintain consent for existing consented users and newly consented users.

If the user does not consent to either vendor Prebid will not load.

CMP update here: guardian/consent-management-platform#921

Copy link

changeset-bot bot commented Feb 9, 2024

🦋 Changeset detected

Latest commit: 6ec7907

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@guardian/commercial Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Jakeii
Copy link
Member

Jakeii commented Feb 9, 2024

Should be able to enable the playwright test again in this PR too right?

@arelra arelra added the [beta] @guardian/commercial Add this label to publish an @guardian/commercial beta npm release from a PR label Feb 9, 2024
Copy link
Contributor

github-actions bot commented Feb 9, 2024

🚀 0.0.0-beta-20240209125551 published to npm as a beta release

Copy link
Contributor

github-actions bot commented Feb 9, 2024

🚀 0.0.0-beta-20240209141457 published to npm as a beta release

@arelra arelra added [beta] @guardian/commercial Add this label to publish an @guardian/commercial beta npm release from a PR and removed [beta] @guardian/commercial Add this label to publish an @guardian/commercial beta npm release from a PR labels Feb 9, 2024
Copy link
Contributor

github-actions bot commented Feb 9, 2024

🚀 0.0.0-beta-20240209161454 published to npm as a beta release

arelra and others added 4 commits February 12, 2024 13:09
Co-authored-by: Akinsola Lawanson <106528085+akinsola-guardian@users.noreply.github.com>
arelra and others added 2 commits February 12, 2024 13:13
This brings in the new custom Prebid vendor

Co-authored-by: Akinsola Lawanson <106528085+akinsola-guardian@users.noreply.github.com>
@arelra arelra marked this pull request as ready for review February 12, 2024 13:33
@arelra arelra requested a review from a team as a code owner February 12, 2024 13:33
@arelra arelra merged commit 2e1a72f into main Feb 12, 2024
10 checks passed
@arelra arelra deleted the ravi/prebid-dual-vendors branch February 12, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[beta] @guardian/commercial Add this label to publish an @guardian/commercial beta npm release from a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants