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

Notification identifiers are not unique #69

Open Antontelesh opened 6 years ago

Antontelesh commented 6 years ago

Currently notification ids are based on current time only.

When an app triggers two notifications at the same time, one of them is not shown. I believe, it'd be better to generate ids using the nanoid package or at least (if you don't want to introduce new dependency) make them string-based and concatenate a unique counter.