Skip to content

Commit

Permalink
feat(surveys): Link type surveys no longer require a url and can be u…
Browse files Browse the repository at this point in the history
…sed as a CTA (#19320)

* allow for link surveys to not require a link

* make it obvious link type surveys can also double as notification

* fix confirmation display

* Update UI snapshots for `chromium` (2)

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
  • Loading branch information
liyiy and github-actions[bot] authored Dec 14, 2023
1 parent 9f646ba commit 7cc6e81
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 5 deletions.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion frontend/src/scenes/surveys/SurveyEditQuestionRow.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ export function SurveyEditQuestionGroup({ index, question }: { index: number; qu
),
},
{
label: 'Link',
label: 'Link/Notification',
value: SurveyQuestionType.Link,
tooltip: () => (
<BaseAppearance
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/scenes/surveys/constants.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ export const defaultSurveyFieldValues = {
},
],
appearance: {
thankYouMessageHeader: 'Redirecting ...',
thankYouMessageHeader: 'Thank you for your feedback!',
},
},
[SurveyQuestionType.Rating]: {
Expand Down
3 changes: 0 additions & 3 deletions frontend/src/scenes/surveys/surveyLogic.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -676,9 +676,6 @@ export const surveyLogic = kea<surveyLogicType>([
name: !name && 'Please enter a name.',
questions: questions.map((question) => ({
question: !question.question && 'Please enter a question.',
...(question.type === SurveyQuestionType.Link
? { link: !question.link && 'Please enter a url for the link.' }
: {}),
...(question.type === SurveyQuestionType.Rating
? {
display: !question.display && 'Please choose a display type.',
Expand Down

0 comments on commit 7cc6e81

Please sign in to comment.