docs(ads): clarify adding platforms to active syncs and segment preservation on delete - #2350
Merged
Merged
Conversation
…rvation on delete - Replace incorrect "Expand your reach" claim (legacy-only feature) with accurate answer: destination platforms are locked once a sync is active - Add explicit note that deleting an ad sync does NOT delete the underlying segment - Remove confusing "use Deactivate instead" paragraph (no such button exists in new Ads UI) - Fix "deactivate or permanently delete" → "delete" in field mapping and too-small FAQs - Add segment-preservation note in "too small for campaigns" troubleshooting section Source: Intercom ticket 215475650890846 (DOC-2105)
Contributor
george-dilthey
marked this pull request as ready for review
August 26, 2026 22:03
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
Four targeted edits to
docs/clay-ads.md:"Can I add another ad platform to an existing Ad Sync?" — Replaced the incorrect "Expand your reach" claim (that section only exists in the legacy table-based ad sync experience
AdAudienceSidebar, not the new Audience Ads 2.0) with the accurate answer: destination platforms are locked once a sync is active; the workaround is to delete and recreate with all destinations configured from the start. Added note that deleting the sync does not affect the underlying audience segment."Can I permanently delete an Ad Sync?" — Added a sentence clarifying that deleting an ad sync does not delete the underlying audience segment. Removed the paragraph referencing "Deactivate" as a separate UI option distinct from "Delete" — the only available action in the new Ads experience is Delete (confirmed in
AdSyncsHomepageTable.tsx: soleDropdownItemlabeled "Delete")."Can I edit the field mapping after setting up an Ad Sync?" — Removed the reference to a "Deactivate" option (doesn't exist in the new Ads UI) and changed "deactivate or permanently delete" to just "delete". Added that deletion doesn't affect the underlying segment.
"Why does my ad audience show 'too small for use in campaigns'?" — Changed "deactivate the current sync" to "delete the current sync" and updated "Deactivation is permanent" to "Deletion is permanent". Added note that deleting the sync doesn't affect the underlying segment.
Why
Intercom ticket #215475650890846: Genesis-Vincent Velarde (Enterprise customer, Tealium) had 3 existing ad syncs going to LinkedIn and wanted to add Google Ads as a destination. The docs said "Expand your reach" would appear — but that section is only in the deprecated legacy table-based ad sync sidebar (
AdAudienceSidebar/components/AddProviderSection.tsx), not in the new Audience Ads 2.0 experience (AdSyncs/). The customer was also unable to find a "deactivate" button (the doc described "Deactivate" as a separate option from "Delete"), and worried that clicking "Delete" would remove their audience segment (the doc said nothing about segment preservation). Support rep Ariana explicitly committed to updating the docs in the conversation.Source link(s)
apps/api/v3/ad-audiences/domain/utils.ts:274-279—validateAdSyncEditablethrows for any non-draft sync. PASS.apps/frontend/src/components/WorkspaceHomepage/AdAudiencesHomepage/AdSyncsHomepage/AdSyncsHomepageTable.tsx:55-68— soleDropdownItemis "Delete". PASS.apps/api/v3/ad-audiences/services/ad-sync.service.ts:311-326—tearDownAdSyncsoft-deletes the sync record only; no segment deletion in the call chain. PASS.apps/frontend/src/components/AdAudienceSidebar/components/AddProviderSection.tsx:115— only inAdAudienceSidebar(deprecated table ads, confirmed by deprecation banner inAdAudienceHeader.tsx).AdSyncs/has no equivalent component.Uncertainties
None. All claims verified against code via the Docsbot Verifier sub-agent and corroborated by the product engineer in the #proj-ads Slack thread.
Related observations
docs/clay-ads.md. None touch the four sections I'm modifying. PR docs(clay-ads): add EEA consent, platform fields, delete-recreate behavior, match rate clarification #2251 adds content about what happens to the ad platform's audience list when you delete and recreate (a different topic than Clay segment preservation), so reviewers should verify the two don't create a contradiction when both merge.#will-i-be-charged-again-if-i-deactivate-and-recreate-an-ad-syncwhich is referenced from multiple places in the doc.FED-3221("unable to add destinations to an existing ad sync") was linked to this Intercom conversation — indicates the product team is tracking this limitation.Closes DOC-2105