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

onActionClick doesn't work #15

Closed steplerbox closed 8 years ago

steplerbox commented 8 years ago

Pass props to Notif in Notifs component: https://github.com/indexiatech/re-notif/blob/master/src/components/Notifs.js#L16 <Notif key={getter(notif, 'id')} message={getter(notif, 'message')} kind={getter(notif, 'kind')} componentClassName={componentClassName} CustomComponent={CustomComponent} onActionClick={onActionClick} actionLabel={actionLabel} />

Check onClick in Notif component: https://github.com/indexiatech/re-notif/blob/master/src/components/Notif.js#L16 if (this.props.onClick) {

How onClick can appear in Notif props?

NameFILIP commented 8 years ago

yeah, I think it should be 

if (this.props.onActionClick) {
  this.props.onActionClick();
kylecesmat commented 8 years ago

Fixed by #19 - going to close this issue