-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(surveys): enable button text per question instead #18623
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense, looks good 👍
Not sure if just missing it, or will come in a follow up, but I don't yet see how someone can add the button text in the UI?
📸 UI snapshots have been updated3 snapshot changes in total. 0 added, 3 modified, 0 deleted:
Triggered by this commit. |
f1e3c07
to
fe283aa
Compare
📸 UI snapshots have been updated3 snapshot changes in total. 0 added, 3 modified, 0 deleted:
Triggered by this commit. |
66a5566
to
2731116
Compare
📸 UI snapshots have been updated4 snapshot changes in total. 0 added, 4 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
* add button text field to question * clean up last traces of new surveys with submitButtonText * button text * fix empty string bug * clean up * linter
Problem
We only had 1 field to set up a "submit button text" value, which doesn't work for multiple question surveys.
Changes
PostHog/posthog-js#893
👉 Stay up-to-date with PostHog coding conventions for a smoother review.
How did you test this code?