Skip to content

Commit

Permalink
fix(surveys): favor "properties" to "user properties", given that we …
Browse files Browse the repository at this point in the history
…can target by user and group properties (#26991)
  • Loading branch information
dmarticus authored Dec 17, 2024
1 parent 1847e4a commit 54e483b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions cypress/e2e/surveys.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand Down Expand Up @@ -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()
Expand Down Expand Up @@ -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 })
Expand Down
6 changes: 3 additions & 3 deletions frontend/src/scenes/surveys/SurveyEdit.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -719,7 +719,7 @@ export default function SurveyEdit(): JSX.Element {
</>
)}
</LemonField>
<LemonField.Pure label="User properties">
<LemonField.Pure label="Properties">
<BindLogic
logic={featureFlagLogic}
props={{ id: survey.targeting_flag?.id || 'new' }}
Expand All @@ -743,7 +743,7 @@ export default function SurveyEdit(): JSX.Element {
setSurveyValue('remove_targeting_flag', false)
}}
>
Add user targeting
Add property targeting
</LemonButton>
)}
{targetingFlagFilters && (
Expand Down Expand Up @@ -772,7 +772,7 @@ export default function SurveyEdit(): JSX.Element {
setSurveyValue('remove_targeting_flag', true)
}}
>
Remove all user properties
Remove all property targeting
</LemonButton>
</>
)}
Expand Down

0 comments on commit 54e483b

Please sign in to comment.