Skip to content

Consent on Profile #7211

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

Merged
merged 24 commits into from
May 15, 2025
Merged

Consent on Profile #7211

merged 24 commits into from
May 15, 2025

Conversation

forstisabella
Copy link
Contributor

@forstisabella forstisabella commented Nov 8, 2024

Proposed changes

Update Consent in Unify page to be Consent on Profile, add net-new content

Merge timing

TBD - requires PM approval

Related issues (optional)

#5738

DOC-792

Copy link

netlify bot commented Nov 8, 2024

Deploy Preview for segment-docs ready!

Name Link
🔨 Latest commit 6bb44d9
🔍 Latest deploy log https://app.netlify.com/sites/segment-docs/deploys/672e8252259409000894f73e
😎 Deploy Preview https://deploy-preview-7211--segment-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Nov 8, 2024

Deploy Preview for segment-docs ready!

Name Link
🔨 Latest commit dacae9d
🔍 Latest deploy log https://app.netlify.com/sites/segment-docs/deploys/681bb8ccb3f994000834ec5b
😎 Deploy Preview https://deploy-preview-7211--segment-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@forstisabella forstisabella changed the base branch from develop to master May 12, 2025 18:06
@forstisabella forstisabella marked this pull request as ready for review May 12, 2025 18:07
@forstisabella forstisabella requested review from pwseg and a team as code owners May 12, 2025 18:07
Events without the consent object will continue to flow to destinations without consent enforcement.
Events without the consent object will continue to flow to destinations without consent enforcement.

Segment events without a context or consent object or that have an empty context or consent object do not impact [consent preferences on the Profile](/docs/privacy/consent-management/consent-in-unify) and these events will flow to all destinations.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Segment events without a context or consent object or that have an empty context or consent object do not impact [consent preferences on the Profile](/docs/privacy/consent-management/consent-in-unify) and these events will flow to all destinations.
Segment events without a context or consent object or that have an empty context or consent object do not impact [consent preferences on the Profile](/docs/privacy/consent-management/consent-in-unify) and these events will flow to all destinations.

There's a lot of branching with these or clauses, any way we could simplify this sentence? Maybe something like "Events that don't include a context object, a consent object, or that include them but leave them empty, don’t affect profile-level consent preferences and will be sent to all destinations."

Or maybe "Segment events that are missing a context or consent object (or include them without any values) will still flow to all destinations and won't update consent preferences on the Profile."

@forstisabella forstisabella merged commit c49dbb8 into master May 15, 2025
2 checks passed
@forstisabella forstisabella deleted the consent-on-profile branch May 15, 2025 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants