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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rewrite PollEditor to React-final-form #3591
Conversation
e7a48d5
to
6b160fc
Compare
6b160fc
to
1081d59
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
import type { | ||
fieldArrayMetaPropTypes, | ||
fieldArrayFieldsPropTypes, | ||
} from 'redux-form'; | ||
} from 'react-final-form'; |
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.
react-final-form
does not export these types. However, I don't seem to find them in redux-form
neither ... Has this ever worked?
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.
Probably not 馃槙
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.
Probably existed in the flow type definitions
1081d59
to
b093827
Compare
b093827
to
a9862d1
Compare
a9862d1
to
8b07442
Compare
Add frontend validation for required fields in the poll editor
Description
Literally the title
Result
No visual changes 馃
Testing
Resolves ABA-56