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

chore: Enable GPP Multi-State Privacy String (MSPS) with U.S. Privacy #806

Merged
merged 32 commits into from
Jan 29, 2024

Conversation

akinsola-guardian
Copy link
Contributor

@akinsola-guardian akinsola-guardian commented Aug 18, 2023

What does this change?

IAB is deprecating US privacy string - at the end of September and moving to the multi-state privacy agreement.

Google has said they will continue to support US Privacy String so not all vendors have signed up to use or accept the multi-state privacy string and have said they will continue to use the US privacy string.

To ensure full monetisation:

SP have a back up solution, Combination of the two (GPP and US Privacy String), so we will still be able to monetise those vendors for both frameworks

We need to implement BOTH:

MSPS (Multi-State Privacy String; GPP, Global Privacy Platform)

&

US privacy string (Current, soon to be legacy but still being supported)

https://trello.com/c/6qAHXGHq

https://docs.sourcepoint.com/hc/en-us/articles/18007731422099-Enable-GPP-Multi-State-Privacy-String-MSPS-with-U-S-Privacy-CCPA-solution

Why?

Sourcepoint recommended change

@akinsola-guardian akinsola-guardian requested a review from a team as a code owner August 18, 2023 10:56
@akinsola-guardian akinsola-guardian changed the title Al update ccpa stub chore: Enable GPP Multi-State Privacy String (MSPS) with U.S. Privacy Aug 18, 2023
Copy link
Contributor

@kenoir kenoir left a comment

Choose a reason for hiding this comment

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

Tricksy! Looks good to me.

Copy link

changeset-bot bot commented Dec 27, 2023

🦋 Changeset detected

Latest commit: e13a188

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

This PR includes changesets to release 1 package
Name Type
@guardian/consent-management-platform Minor

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

Copy link
Contributor

github-actions bot commented Jan 11, 2024

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
92.7% (+0.25% 🔼)
254/274
🟢 Branches
83.48% (+0.29% 🔼)
96/115
🟢 Functions 90% 63/70
🟢 Lines
92.48% (+0.29% 🔼)
246/266

Test suite run success

328 tests passing in 16 suites.

Report generated by 🧪jest coverage report action from e13a188

Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

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

@@ -97,5 +98,9 @@ declare global {
callback: (tcData: TCData, success: boolean) => void,
vendorIDs?: number[],
) => void;
__gpp?: (
command: string,
callback: (gppData: any, success: boolean) => void,
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be an 'any' type?

Copy link
Contributor

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

Copy link
Contributor

@sookburt sookburt left a comment

Choose a reason for hiding this comment

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

We did some extra testing of the resulting GPP string and that the Privacy Manager appears, it's looking good! 🎉

@akinsola-guardian akinsola-guardian merged commit 60b7d15 into main Jan 29, 2024
16 checks passed
@akinsola-guardian akinsola-guardian deleted the al-update-ccpa-stub branch January 29, 2024 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants