docs(clay-ads-compliance-best-practices): add exclusion FAQ, hashing FAQ, two best-practice bullets, update subtitle - #2289
Open
george-dilthey wants to merge 3 commits into
Conversation
…FAQ, two best-practice bullets, and note on Enhanced matching
Contributor
… drop unverified Enhanced matching sentence
… house content rules
george-dilthey
marked this pull request as ready for review
August 26, 2026 14:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Applied four upstream deltas from the Webflow
collection_item_publishedwebhook todocs/clay-ads-compliance-best-practices.md:One upstream delta was not applied: the sentence "That covers the enrichments most ad audiences rely on, including additional emails and
Enhanced matchingresults." — dropped because the CPJ exclusion logic it describes is gated by theenable-ads-cpj-exclusionEppo flag (defaultValue: false, only tested on one workspace as of April 2026, no GA announcement found). See Uncertainties.Local-only enrichments preserved: "Recurring syncs run every 3 days" in the Best Practices opt-outs bullet and opt-outs FAQ answer (not in either upstream version).
Why
Triggered by a Webflow
collection_item_publishedwebhook for https://university.clay.com/docs/clay-ads-compliance-best-practices.Source link(s)
apps/api/v3/ad-audiences/domain/ad-audience-segment-export-context.ts:104–148apps/api/v3/ad-audiences/domain/google/google-field-validators.ts:48–137apps/api/v3/ad-audiences/domain/meta/meta-field-validators.ts:25–102apps/api/v3/ad-audiences/domain/linkedin/linkedin-field-validators.ts:102–104libs/shared/src/ad-audiences/all-field-names.ts:138+apps/api/v3/ad-audiences/domain/bing/bing-field-validators.ts:13–31apps/api/v3/ad-audiences/domain/ad-sync-cpj-exclusion-filters.ts:20–26(EppoFeatureFlags.EnableAdsCpjExclusion,defaultValue: false)Uncertainties
CPJ exclusion /
enable-ads-cpj-exclusionEppo flag: Both the new "Enhanced matching" sentence and, to a lesser extent, the exclusion audiences FAQ depend on the CPJ exclusion filtering logic gated byenable-ads-cpj-exclusion(defaultValue: false). The Slack#team-adsthread from April 21 2026 shows the flag was being tested on a single workspace; no GA/all-customers rollout announcement was found. I dropped the "Enhanced matching" sentence entirely (too specific to the gated feature). The exclusion audiences FAQ is included because it describes the general architecture (filtering happens at build time, not based on audience intent), which is accurate when the feature is on — but a reviewer should confirm whether this Eppo flag has since been rolled out broadly before merging.Related observations
The upstream diff also included a sentence referencing "Enhanced matching" as a covered enrichment type. That label does not appear in the codebase; it may be a product UI term worth verifying with the Ads team before adding to docs.
Closes DOC-2036