indexiatech / re-notif

Redux & React based Notifications center.

Home Page:http://indexiatech.github.io/re-notif

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Notification identifiers are not unique

Antontelesh opened this issue · comments

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.