-
Notifications
You must be signed in to change notification settings - Fork 9
Update forms to use custom Input ui component #281
Conversation
See https://fb.me/react-warning-keys for more information.
See http://redux-form.com/5.3.1/#/api/props for more information.
See redux-form/redux-form#1249 for more information.
valid, visited, asyncValidating, autofocus, ...validProps } = props; | ||
const { | ||
active, | ||
autofill, |
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.
What is the purpose of removing asyncValidating
from this list?
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.
That's because it's useless since it's not part of the list of props that redux-form
provides inside the fields
object.
Code 👍 |
Product 👍 |
* @returns {object} props filtered | ||
* https://github.com/erikras/redux-form/issues/1249 | ||
* | ||
* This function should be removed once redux-form is upgraded to version 6. |
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.
It seems v6 won't fix this, it just provides a cleaner syntax to create input components.
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.
Oh ok it has this input
prop with all the valid properties
This pull request fixes https://github.com/Automattic/delphin/issues/249 to make the forms on the
Login
,Signup
, andVerify User
pages more consistent with the rest of the forms. More specifically, it makes sure the custom<Input>
component is used in every appropriate places so validation errors are correctly displayed. This pull request also updates the code to address several React warnings that were generated by theredux-form
library.Testing instructions
You should keep an eye on your browser's console for possible warning and error messages:
git checkout update/forms
and start your server, or open a live branchSearch
pageAdditional notes
Note the following warning is still displayed in the console when inline validation is triggered:
This is something that has been addressed in facebook/react#6471 and that will be available in React 15.3.0.
Reviews
@Automattic/sdev-feed