-
Notifications
You must be signed in to change notification settings - Fork 2
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
Test consent or pay #1783
base: main
Are you sure you want to change the base?
Test consent or pay #1783
Conversation
🦋 Changeset detectedLatest commit: 6abbc67 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 |
Tip Once this PR is ready to go, add the This saves us a lot of money by not running the tests before we need them. |
51c9b24
to
fbfd618
Compare
🐥 |
🐥 |
08286ba
to
334c67d
Compare
🐥 |
🐥 |
🐥 |
🐥 |
🐥 |
🐥 |
59c2dbf
to
281c7d8
Compare
🐥 |
🐥 |
Note The following canaries were published to NPM: 🐥 |
🐥 |
🐥 |
🐥 |
framework: ConsentFramework, | ||
countryCode: CountryCode, | ||
subscriber: boolean, | ||
isUserSignedIn: boolean, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not required for subdomains
@@ -92,6 +106,10 @@ const showPrivacyManager = () => { | |||
void initialised.then(UnifiedCMP.showPrivacyManager); | |||
}; | |||
|
|||
const rejectAll = isServerSide |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove rejectAll
if ( | ||
choiceTypeID === SourcePointChoiceTypes.RejectAll && | ||
message_type === 'gdpr' && | ||
!subscriber && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrap in function
🐥 |
What are you changing?
Why?