Skip to content

Conversation

cheton
Copy link
Contributor

@cheton cheton commented Mar 3, 2017

We usually pass a string (or React node) instead of function to the "emptyText" prop, it will be great if the "emptyText" prop can accept either node or function type.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling dd4152d on cheton:master into ** on react-component:master**.

@yesmeck
Copy link
Member

yesmeck commented Mar 6, 2017

Hello @chentony , could you write a test?

@cheton
Copy link
Contributor Author

cheton commented Mar 14, 2017

Done. Thank you.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 005c4d9 on cheton:master into ** on react-component:master**.

@yesmeck yesmeck merged commit 1d08ca9 into react-component:master Mar 14, 2017
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