search

juliancwirko / react-s-alert

Alerts / Notifications for React with rich configuration options
https://www.npmjs.com/package/react-s-alert
MIT License
649 stars 75 forks source link

alert also accepts a react component #19

Closed FredericHeem closed 8 years ago

juliancwirko commented 8 years ago

Hi, this is a very nice idea, but I wonder if it will work in all cases. Have you tested it with components which has more complicated structures? I'll try to find some time to test it.

FredericHeem commented 8 years ago

FYI, here is the component being rendered: https://github.com/FredericHeem/starhackit/blob/master/client/src/app/parts/core/components/notification.js It is working fine on my side.

FredericHeem commented 8 years ago

Any news on merging this PR ? I'm fed up to see the react warnings polluting the console log

juliancwirko commented 8 years ago

Hi, sorry about that, I plan to get back to this library. I hadn't enough time (open source life time ;)). But give me some time and everything will be on the proper way :)