-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Notifications Appearing as duplicates #56
Comments
??? |
I have integrated firebase push in my web app,when a push notification is sent,the notifications appears on nth times the push was send i.e if i send twice then send again,the user will see four notifications with the same message. |
Earlier this morning i encountered this error too how can i fix? |
The duplicate thing is really weird, I don't see any explanation for that why is it happening. The second case where the key is duplicated was reported in the past but I was never able to reproduce it. Id is a |
To reproduce the second error on random ids, send a notification and before the timeout of the first one, send another one.
Works fine when no concurrent notifications.
I managed to fix the duplicates which i guess was being produced by the fact that i was using React.memo on the stateless component.
On 5 Mar 2020 09:48, Rares Mardare <notifications@github.com> wrote:
The duplicate thing is really weird, I don't see any explanation for that why is it happening. The second case where the key is duplicated was reported in the past but I was never able to reproduce it. Id is a Math.random randomisation, I doubt it's generating twice the same result for same timestamp. Will further investigate but atm I have no clues.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#56?email_source=notifications&email_token=AFMTXN3DJBMVKBZRRZ2MPY3RF5DKHA5CNFSM4LAOPOY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEN35UUQ#issuecomment-595057234>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AFMTXNZV6SY3YP2G7EVRCA3RF5DKHANCNFSM4LAOPOYQ>.
|
I changed the way we were generating the id, I suspect the bug was due to the random function. Fix was done in Please check and let me know if it's working for you now. |
I will update tomorrow and revert on the same. On 14 Mar 2020 19:15, Rares Mardare <notifications@github.com> wrote:
I changed the way we were generating the id, I suspect the bug was due to the random function. Fix was done in 2.4.0 and id will point to an increment rather than a random.
Please check and let me know if it's working for you now.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.
|
The text was updated successfully, but these errors were encountered: