Skip to content

Add a support for React 16; fix a warning regarding prop-types; update tributejs - #7

Open
phyper wants to merge 4 commits into
BolsteDev:masterfrom
phyper:master
Open

Add a support for React 16; fix a warning regarding prop-types; update tributejs#7
phyper wants to merge 4 commits into
BolsteDev:masterfrom
phyper:master

Conversation

@phyper

@phyper phyper commented Nov 30, 2017

Copy link
Copy Markdown

No description provided.

@emwalker

Copy link
Copy Markdown

@imontiel, any thoughts on this PR?

@emwalker

emwalker commented Apr 18, 2018

Copy link
Copy Markdown

@phyper I'm seeing a console error which I think goes back to this line:

https://github.com/phyper/react-tribute/blob/master/src/Tribute.js#L19

I'm on React 15, and I think isRequired is needed rather than required. Is this something that works in later versions of PropTypes?

@phyper

phyper commented Apr 18, 2018

Copy link
Copy Markdown
Author

Agree, it should be isRequired instead of required.

@mrsweaters

Copy link
Copy Markdown

Related: #11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants