search

indexiatech / re-notif

Redux & React based Notifications center.
http://indexiatech.github.io/re-notif
BSD 3-Clause "New" or "Revised" License
148 stars 45 forks source link

Change `message: PropTypes.string.isRequired` #38

Closed nixypanda closed 8 years ago

nixypanda commented 8 years ago

Hi great job with the component but I was wondering why message is restricted to only string (in Notif.js). Shouldn't any react-compoent be vaild here?

kylecesmat commented 8 years ago

Not in this case - The default notification component is meant to display a message, and including a component in an action isn't a great approach. You should use the customComponent prop to pass your own component.