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
General changes to surveys #3677
Conversation
Add a "cancel" button to the editor and a "back" navigator. Other than that just styling changes.
a9f0f24
to
f17d14d
Compare
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.
LGTM
Types could be better, otherwise just small comments
updateRelativeIndexes: (arg0: number, arg1: number, arg2: Fields) => void; | ||
fields: Fields; | ||
updateRelativeIndexes: (arg0: number, arg1: number, arg2: any) => void; | ||
fields: any; |
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.
😢
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.
These are so difficult to type 😩
type State = { | ||
templatePickerOpen: boolean; | ||
templateTypeSelected: string; | ||
initialValues: Record<string, any>; |
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.
😢
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.
Do you know how big that object is? I'm not typing that
<ul className={styles.questions} key="questions"> | ||
{fields.map((question, i) => ( | ||
<Question | ||
key={i} |
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.
🤔
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.
Not me
f17d14d
to
8fca652
Compare
8fca652
to
bee93d3
Compare
Description
Minor changes to surveys pages
Add a "cancel" button to the editor and a "back" navigator.
Other than that just styling changes.
Rewrite survey editor to a functional component
Migrate survey editor to react-final-form
Testing
The changes to the
<Field />
component has been checked with fields inredux-form
forms using theonChange
prop.Editing works for the three different options; radio buttons, check boxes and text areas.
Related to ABA-54 (
redux-form
is still used on the actual survey form)