Skip to content

Commit

Permalink
fix(surveys): clearer user property names
Browse files Browse the repository at this point in the history
  • Loading branch information
liyiy committed Oct 26, 2023
1 parent 559ee24 commit 8e60433
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/__tests__/extensions/surveys.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ describe('survey display logic', () => {
$survey_name: 'Test survey 1',
sessionRecordingUrl: undefined,
$set: {
'$survey_dismiss/testSurvey1': true,
'$survey_dismissed/testSurvey1': true,
},
})
expect(localStorage.getItem(`seenSurvey_${mockSurveys[0].id}`)).toBe('true')
Expand Down
4 changes: 2 additions & 2 deletions src/extensions/surveys.ts
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ export const createOpenTextOrLinkPopup = (
$survey_response: surveyQuestionType === 'open' ? e.target.survey.value : 'link clicked',
sessionRecordingUrl: posthog.get_session_replay_url?.(),
$set: {
[`$survey_response/${survey.id}`]: true,
[`$survey_responded/${survey.id}`]: true,
},
})
if (surveyQuestionType === 'link') {
Expand Down Expand Up @@ -461,7 +461,7 @@ export const addCancelListeners = (
$survey_id: surveyId,
sessionRecordingUrl: posthog.get_session_replay_url?.(),
$set: {
[`$survey_dismiss/${surveyId}`]: true,
[`$survey_dismissed/${surveyId}`]: true,
},
})
})
Expand Down

0 comments on commit 8e60433

Please sign in to comment.