docs(clay-ads): broaden beta note from Sync panel to Clay Ads feature#1504
Open
george-dilthey wants to merge 1 commit into
Open
docs(clay-ads): broaden beta note from Sync panel to Clay Ads feature#1504george-dilthey wants to merge 1 commit into
george-dilthey wants to merge 1 commit into
Conversation
Contributor
This was referenced Jul 13, 2026
Merged
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
Updated a single note in
docs/clay-ads.md: replaced "The Sync panel is currently in beta" with "Clay Ads is currently in beta". The previous phrasing scoped the beta status to one specific UI panel; the deployed commit adds aBetaBadgeto the entire Ad Sync top navigation bar (visible on every Ad Sync view), making the broader scope accurate.Why
Deploy
clay-run/clay-base6e59a78...8e4eb05(https://github.com/clay-run/clay-base/actions/runs/29109126961) introduced commit8e4eb050:fix(ads): bad css,remove press that leads to billing page,beta badge (#43418). Among other CSS/UX fixes, this commit adds<BetaBadge />toWorkspaceTopbarNavigatorAdSync.tsx— the breadcrumb/top navigation shown when a user is viewing any Ad Sync. The badge renders unconditionally for all users who have access to the Ad Sync route.The existing doc note said only "The Sync panel" (the right-hand status panel in Step 4) is in beta, which undersells the beta scope: the verifier confirmed the entire Ad Sync route is gated by the
EnableAdsV2Eppo flag (defaultfalse) atapps/frontend/src/routes/WorkspaceRoutes.tsx:214-223, and the BetaBadge appears on the overall feature view, not just one panel within it.Source link(s)
8e4eb050:WorkspaceTopbarNavigatorAdSync.tsx:40adds<BetaBadge />unconditionally (verifier file:line evidence)EppoFeatureFlags.EnableAdsV2 = 'enable-ads-v2', defaultfalse, atlibs/shared/src/EppoFeatureFlags.ts:206— gates the entire Ad Sync routeBetaBadgetooltip:'This feature is only available to Beta users'atapps/frontend/src/components/Badges/BetaBadge.tsx:9EnableAdsV2enabledUncertainties
The
EnableAdsV2flag's exact targeting config lives in the Eppo dashboard (outside the repo). The verifier's Slack context confirms "open beta for Growth plan" as of a July 9 message, consistent with rolling out to Growth and Enterprise as the docs state. No hard uncertainty on the claim itself.Related observations
#proj-adsby Larry Li describes relaxing the GDPR filter so non-US contacts from 1P-origin data (CSV/CRM/DWH) are no longer auto-filtered from LinkedIn/Meta syncs. This is a separate behavioral change from a prior deploy and may warrant a separate doc update toclay-ads-compliance-best-practices.md(which currently says "Today, you're not able to source contacts outside the US for ad targeting" — potentially stale after the July 7 change).EnableAdsV2), which is consistent with the feature still being in controlled beta rollout despite the earlier GA announcement for Enterprise customers.Confidence: medium
Closes DOC-1085