From 54e483b60059bc414f640463a46ad77b57ad9472 Mon Sep 17 00:00:00 2001 From: Dylan Martin Date: Tue, 17 Dec 2024 15:41:50 -0500 Subject: [PATCH] fix(surveys): favor "properties" to "user properties", given that we can target by user and group properties (#26991) --- cypress/e2e/surveys.cy.ts | 6 +++--- frontend/src/scenes/surveys/SurveyEdit.tsx | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/cypress/e2e/surveys.cy.ts b/cypress/e2e/surveys.cy.ts index 604c182a51a18..1680681bf1fb5 100644 --- a/cypress/e2e/surveys.cy.ts +++ b/cypress/e2e/surveys.cy.ts @@ -92,7 +92,7 @@ describe('Surveys', () => { cy.get('.LemonCollapsePanel').contains('Display conditions').click() cy.contains('All users').click() cy.get('.Popover__content').contains('Users who match').click() - cy.contains('Add user targeting').click() + cy.contains('Add property targeting').click() // select the first property cy.get('[data-attr="property-select-toggle-0"]').click() @@ -144,7 +144,7 @@ describe('Surveys', () => { // remove user targeting properties cy.get('.LemonCollapsePanel').contains('Display conditions').click() - cy.contains('Remove all user properties').click() + cy.contains('Remove all property targeting').click() // save cy.get('[data-attr="save-survey"]').eq(0).click() @@ -197,7 +197,7 @@ describe('Surveys', () => { cy.get('.LemonCollapsePanel').contains('Display conditions').click() cy.contains('All users').click() cy.get('.Popover__content').contains('Users who match').click() - cy.contains('Add user targeting').click() + cy.contains('Add property targeting').click() cy.get('[data-attr="property-select-toggle-0"]').click() cy.get('[data-attr="prop-filter-person_properties-0"]').click() cy.get('[data-attr=prop-val]').click({ force: true }) diff --git a/frontend/src/scenes/surveys/SurveyEdit.tsx b/frontend/src/scenes/surveys/SurveyEdit.tsx index ba5f62bcce81b..b25fce787d71f 100644 --- a/frontend/src/scenes/surveys/SurveyEdit.tsx +++ b/frontend/src/scenes/surveys/SurveyEdit.tsx @@ -719,7 +719,7 @@ export default function SurveyEdit(): JSX.Element { )} - + - Add user targeting + Add property targeting )} {targetingFlagFilters && ( @@ -772,7 +772,7 @@ export default function SurveyEdit(): JSX.Element { setSurveyValue('remove_targeting_flag', true) }} > - Remove all user properties + Remove all property targeting )}