-
Notifications
You must be signed in to change notification settings - Fork 26
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
Improve form examples, add errors to FieldErrors, fix TextField #76
Conversation
a5b63eb
to
1bb99ab
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.
No objections. Looks sharp.
isTouched={this.props.formik.touched.inventory} | ||
errors={this.props.formik.errors.inventory} | ||
renderError={key => { | ||
switch (key) { |
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.
Starting to like render callbacks here. At first I was sceptical and wanted s component but they‘re easier on the consumer.
// to accidental loss of data. By keeping the version in the form values | ||
// instead an update of the version in the Apollo Cache does not affect the. | ||
// form. On submission the version from the form values will be used, and the | ||
// user will see the APIs ConcurrentModificationError as expected! |
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.
💯😘
This component will be used to show the errors of all fields.
a5ea9f7
to
3ec1735
Compare
Storybook
Features
Fixes