Skip to content
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

Silence React 15.4 invalid property warnings #80

Merged
merged 2 commits into from
Dec 12, 2016
Merged

Silence React 15.4 invalid property warnings #80

merged 2 commits into from
Dec 12, 2016

Conversation

nathanstitt
Copy link
Contributor

Remove planceHolderChar and formatCharacters from props before setting them on input element.

The specs had console.error stubbed out, which may be why the warnings weren't noticed. This restores the spy after it's used and adds an explicit spec to test the props are removed.

Before passing them to input in order to silence invalid prop warnings
introduced in React 15.4
@esarbanis
Copy link

We really need this fix in our project. Nothing worse that customers complaining about console errors :)

@iamdustan iamdustan merged commit ffae092 into insin:master Dec 12, 2016
@iamdustan
Copy link
Collaborator

thanks!

@nathanstitt
Copy link
Contributor Author

Thanks for the excellent component @iamdustan !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants