Skip to content

Less strict PropTypes for string placeholders to optionally allow React nodes#655

Merged
JedWatson merged 1 commit intoJedWatson:masterfrom
kromit:patch-1
Dec 10, 2015
Merged

Less strict PropTypes for string placeholders to optionally allow React nodes#655
JedWatson merged 1 commit intoJedWatson:masterfrom
kromit:patch-1

Conversation

@kromit
Copy link
Contributor

@kromit kromit commented Dec 9, 2015

Remove the warning without modifying the functionality by using

React.PropTypes.oneOfType([
     React.PropTypes.string,
    React.PropTypes.node
]) 

instead of

React.PropTypes.string,

…ct nodes

Remove the warning without modifying the functionality by using

    React.PropTypes.oneOfType([
         React.PropTypes.string,
        React.PropTypes.node
    ]) 

instead of 

    React.PropTypes.string,
@alyssaBiasi
Copy link
Contributor

FYI similar to #641

JedWatson added a commit that referenced this pull request Dec 10, 2015
Less strict PropTypes for string placeholders to optionally allow React nodes
@JedWatson JedWatson merged commit 791dea7 into JedWatson:master Dec 10, 2015
@JedWatson
Copy link
Owner

Looks great @kromit, thanks! I'm about to merge #641 as well, @alyssaBiasi :)

@kromit kromit deleted the patch-1 branch December 10, 2015 08:15
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.

3 participants