reply on: SN release: PWA push notifications \ stacker news ~meta

0 sats \ 10 replies \ @ek 6 Jul 2023 \ parent \ on: SN release: PWA push notifications meta

🤨 interesting

0 sats \ 9 replies \ @WeAreAllSatoshi 6 Jul 2023

Seems like a timing issue/race condition accessing the zap amount, if I had to guess. I’m guessing something about the push notification pipeline is asynchronous, so that would make sense.

0 sats \ 8 replies \ @ek 6 Jul 2023

Yes, it's async so that could indeed explain why you see 2,3,2

But I don't get why the notifications are not grouped together using the tag property like on Android. Here, it says that Safari on iOS also supports it.

My guess would be there is some difference in the API for replacing notifications.

0 sats \ 7 replies \ @ek 6 Jul 2023

Ohhh, I think I found the error.

Let me guess. If I reply three times to you, you only get a single notification which shows "you have 3 new replies", right?

Let this be the first reply.

0 sats \ 6 replies \ @ek 6 Jul 2023

Second reply.

0 sats \ 5 replies \ @ek 6 Jul 2023

Third reply.

Please tell me you only got one notification haha

If so, I am pretty sure what is causing the bug

0 sats \ 4 replies \ @WeAreAllSatoshi 6 Jul 2023

I got 3 notifications

0 sats \ 3 replies \ @ek 6 Jul 2023

damn

0 sats \ 2 replies \ @k00b OP 6 Jul 2023

Same. I don't think there's any grouping happening on iOS.

view replies