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

Consent or pay #1901

Open

akinsola-guardian wants to merge 23 commits into main from al-consent-or-pay

+510 −70

Contributor

akinsola-guardian commented Jan 20, 2025 •

edited

Loading

What are you changing?

Extending the CMP.init to take in two new variables, useNonAdvertisedList and isUserSignedIn
Using Sourcepoint Subdomain property if useNonAdvertisedList is true
Merging user consent from Advertising to Non-Advertising vendor list
Redirecting user to Support when clicking Reject for Consent or Pay users
Extending __tcfapi to postCustomConsent as part of merging user consent process.

Why?

Consent Or Pay

akinsola-guardian added 4 commits

January 20, 2025 11:37


          Update types to include isCorp, isUserSignedIn, useNonAdvertisedList

c549898


          Update Config for Property Id's and Href

1df9542


          Use isUserSignedIn and useNonAdvertisedList in CMP.init

0067a05


          Create consent or pay function and test

800c6a1

changeset-bot bot commented Jan 20, 2025 •

edited

Loading

🦋 Changeset detected

Latest commit: 5cf6d26

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

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

Contributor

github-actions bot commented Jan 20, 2025

Tip

Once this PR is ready to go, add the run_chromatic label to run the Chromatic tests.

This saves us a lot of money by not running the tests before we need them.

github-actions bot added 📦 npm @guardian/libs labels

akinsola-guardian added 6 commits

January 20, 2025 13:33


          Add type to import

716039f


          Update to account for useNonAdvertisedList

86e8645


          Change to window.open and use isConsentOrPay

edd1fdd


          Add isConsentOrPay to check

2a9a39b


          Update test page

ae77c08


          Created function mergeUserConsent

c7b0f96

akinsola-guardian force-pushed the al-consent-or-pay branch from 0fea6c2 to c7b0f96 Compare

January 20, 2025 15:35

akinsola-guardian added the 🐥 Canaries label


          Create breezy-drinks-live.md

b56c2b7

Contributor

github-actions bot commented Jan 23, 2025

Note

The following canaries were published to NPM:

🐥

tomrf1 mentioned this pull request

WIP - consent or pay guardian/dotcom-rendering#13199

Draft


          Check if user is a participant of ConsentOrPayBanner

88aa907

Contributor

github-actions bot commented Jan 28, 2025

Note

The following canaries were published to NPM:

🐥

akinsola-guardian added 3 commits

February 3, 2025 11:41


          Add new api function to postCustomConsent

55f4b44


          Create function to mergeVendorList

b8c7fe7


          Use localstorage to log if a user has had their vendor list merged

cda7521

Contributor

github-actions bot commented Feb 3, 2025

Note

The following canaries were published to NPM:

🐥


          Fix tests

91395bd

Contributor

github-actions bot commented Feb 4, 2025

Note

The following canaries were published to NPM:

🐥

Contributor

github-actions bot commented Feb 4, 2025

Note

The following canaries were published to NPM:

🐥


          Update isInConsentOrPay

4ecb9f9

akinsola-guardian force-pushed the al-consent-or-pay branch from 410cb48 to 4ecb9f9 Compare

February 4, 2025 15:50

Contributor

github-actions bot commented Feb 4, 2025

Note

The following canaries were published to NPM:

🐥


          Update function to load privacy manager

51e3da4

Contributor

github-actions bot commented Feb 5, 2025

Note

The following canaries were published to NPM:

🐥


          Merge branch 'main' into al-consent-or-pay

a79e694

Contributor

github-actions bot commented Feb 5, 2025

Note

The following canaries were published to NPM:

🐥


          Update ophan message to include Consent or Pay banner

2c7642b

Contributor

github-actions bot commented Feb 5, 2025

Note

The following canaries were published to NPM:

🐥

Contributor

github-actions bot commented Feb 7, 2025

Note

The following canaries were published to NPM:

🐥

akinsola-guardian force-pushed the al-consent-or-pay branch from beaea12 to e99c21e Compare

February 7, 2025 17:28


          Update property name

86df823

akinsola-guardian force-pushed the al-consent-or-pay branch from e99c21e to 86df823 Compare

February 7, 2025 17:31

Contributor

github-actions bot commented Feb 7, 2025

Note

The following canaries were published to NPM:

🐥


          Only redirect users for users in Consent or Pay AB test

3dd58c6

Contributor

github-actions bot commented Feb 10, 2025

Note

The following canaries were published to NPM:

🐥

akinsola-guardian marked this pull request as ready for review

February 10, 2025 10:12

akinsola-guardian requested review from a team as code owners

February 10, 2025 10:12


          Merge branch 'main' into al-consent-or-pay

5cf6d26

Contributor

github-actions bot commented Feb 10, 2025

Note

The following canaries were published to NPM:

🐥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐥 Canaries @guardian/libs 📦 npm