Skip to content

Commit c0dafbc

Browse files
authored
chore(customers): Gate customer analytics behind early access feature flag (#42628)
1 parent 8f3121a commit c0dafbc

File tree

6 files changed

+13
-12
lines changed

6 files changed

+13
-12
lines changed

frontend/src/lib/components/TopBarSettingsButton/topBarSettingsButtonLogic.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,15 +86,15 @@ describe('topBarSettingsButtonLogic', () => {
8686
it('returns other setting section IDs regardless of CRM feature flag state', async () => {
8787
router.actions.push(urls.persons())
8888
featureFlagLogic.actions.setFeatureFlags([], {
89-
[FEATURE_FLAGS.CRM_ITERATION_ONE]: false,
89+
[FEATURE_FLAGS.CUSTOMER_ANALYTICS]: false,
9090
})
9191

9292
await expectLogic(logic).toMatchValues({
9393
loadedSceneSettingsSectionId: 'environment-product-analytics',
9494
})
9595

9696
featureFlagLogic.actions.setFeatureFlags([], {
97-
[FEATURE_FLAGS.CRM_ITERATION_ONE]: true,
97+
[FEATURE_FLAGS.CUSTOMER_ANALYTICS]: true,
9898
})
9999

100100
await expectLogic(logic).toMatchValues({

frontend/src/lib/constants.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -237,10 +237,9 @@ export const FEATURE_FLAGS = {
237237
ENDPOINTS: 'embedded-analytics', // owner: @sakce #team-clickhouse
238238
SUPPORT_FORM_IN_ONBOARDING: 'support-form-in-onboarding', // owner: @joshsny
239239
CRM_ITERATION_ONE: 'crm-iteration-one', // owner: @arthurdedeus #team-customer-analytics
240-
CRM_USAGE_METRICS: 'crm-usage-metrics', // owner: @arthurdedeus #team-customer-analytics
241240
TOGGLE_PROPERTY_ARRAYS: 'toggle-property-arrays', // owner: @arthurdedeus #team-customer-analytics
242241
DWH_JOIN_TABLE_PREVIEW: 'dwh-join-table-preview', // owner: @arthurdedeus #team-customer-analytics
243-
CUSTOMER_ANALYTICS: 'customer-analytics', // owner: @arthurdedeus #team-customer-analytics
242+
CUSTOMER_ANALYTICS: 'customer-analytics-roadmap', // owner: @arthurdedeus #team-customer-analytics
244243
SETTINGS_SESSIONS_V2_JOIN: 'settings-sessions-v2-join', // owner: @robbie-c #team-web-analytics
245244
SESSIONS_EXPLORER: 'sessions-explorer', // owner: @jabahamondes #team-web-analytics
246245
SAVE_INSIGHT_TASK: 'save-insight-task', // owner: @joshsny

frontend/src/queries/nodes/DataTable/DataTable.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,7 @@ export function DataTable({
174174

175175
const canUseWebAnalyticsPreAggregatedTables = useFeatureFlag('SETTINGS_WEB_ANALYTICS_PRE_AGGREGATED_TABLES')
176176
const hasCrmIterationOneEnabled = useFeatureFlag('CRM_ITERATION_ONE')
177+
const hasCustomerAnalyticsEnabled = useFeatureFlag('CRM_ITERATION_ONE')
177178
const usedWebAnalyticsPreAggregatedTables =
178179
canUseWebAnalyticsPreAggregatedTables &&
179180
response &&
@@ -734,7 +735,8 @@ export function DataTable({
734735
].filter((x) => !!x)
735736

736737
const secondRowRight = [
737-
sourceFeatures.has(QueryFeature.linkDataButton) && hasCrmIterationOneEnabled ? (
738+
sourceFeatures.has(QueryFeature.linkDataButton) &&
739+
(hasCrmIterationOneEnabled || hasCustomerAnalyticsEnabled) ? (
738740
<ViewLinkButton tableName="groups" />
739741
) : null,
740742
(showColumnConfigurator || showPersistentColumnConfigurator) &&

frontend/src/scenes/groups/Group.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ export function Group({ tabId }: { tabId?: string }): JSX.Element {
121121
label: <span data-attr="groups-overview-tab">Overview</span>,
122122
content: <GroupOverview groupData={groupData} />,
123123
},
124-
...(featureFlags[FEATURE_FLAGS.CRM_ITERATION_ONE] && groupData.notebook
124+
...(featureFlags[FEATURE_FLAGS.CUSTOMER_ANALYTICS] && groupData.notebook
125125
? [
126126
{
127127
key: GroupsTabType.NOTES,

frontend/src/scenes/groups/Groups.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ export function GroupsScene({ tabId }: { tabId?: string } = {}): JSX.Element {
4646

4747
const { groupsAccessStatus } = useValues(groupsAccessLogic)
4848
const { aggregationLabel } = useValues(groupsModel)
49-
const hasCrmIterationOneEnabled = useFeatureFlag('CRM_ITERATION_ONE')
49+
const hasCustomerAnalyticsEnabled = useFeatureFlag('CUSTOMER_ANALYTICS')
5050

5151
if (groupTypeIndex === undefined) {
5252
throw new Error('groupTypeIndex is undefined')
@@ -79,7 +79,7 @@ export function GroupsScene({ tabId }: { tabId?: string } = {}): JSX.Element {
7979
},
8080
} as QueryContext['columns']
8181
let hiddenColumns = [] as string[]
82-
if (hasCrmIterationOneEnabled) {
82+
if (hasCustomerAnalyticsEnabled) {
8383
columns = getCRMColumns(groupTypeName, groupTypeIndex)
8484
hiddenColumns.push('key')
8585
}
@@ -95,7 +95,7 @@ export function GroupsScene({ tabId }: { tabId?: string } = {}): JSX.Element {
9595
type: 'cohort',
9696
}}
9797
actions={
98-
hasCrmIterationOneEnabled ? (
98+
hasCustomerAnalyticsEnabled ? (
9999
<LemonButton
100100
type="primary"
101101
size="small"
@@ -135,7 +135,7 @@ export function GroupsScene({ tabId }: { tabId?: string } = {}): JSX.Element {
135135
dataAttr="groups-table"
136136
/>
137137

138-
{hasCrmIterationOneEnabled && (
138+
{hasCustomerAnalyticsEnabled && (
139139
<LemonModal
140140
isOpen={saveGroupViewModalOpen}
141141
onClose={() => setSaveGroupViewModalOpen(false)}

frontend/src/scenes/groups/groupsNewLogic.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -213,8 +213,8 @@ export const groupsNewLogic = kea<groupsNewLogicType>([
213213
})),
214214

215215
afterMount(({ props, values }) => {
216-
// Redirect if the CRM feature flag is not enabled
217-
if (!values.featureFlags[FEATURE_FLAGS.CRM_ITERATION_ONE]) {
216+
// Redirect if customer analytics is not enabled
217+
if (!values.featureFlags[FEATURE_FLAGS.CUSTOMER_ANALYTICS]) {
218218
router.actions.push(urls.groups(props.groupTypeIndex))
219219
}
220220
}),

0 commit comments

Comments
 (0)