Tom Jenkinson

Results 110 comments of Tom Jenkinson

Looking at CI maybe this needs to be 2 separate PR's?

> Merged as [8adcfbf](https://github.com/socketio/socket.io/commit/8adcfbfde50679095ec2abe376650cf2b6814325). > > I've made some minor edits and added a few tests. Could you please check if everything makes sense? Thanks! Just this one comment https://github.com/socketio/socket.io/commit/8adcfbfde50679095ec2abe376650cf2b6814325#r146871962

Hey @darrachequesne The problem with retries is that it requires that you wait for the ack, and also with retrying you need to make sure you either are ok processing...

Hey @darrachequesne sometimes when reopening the laptop I'm not seeing the visibility change fire so not sure how reliable it would be Also when sending a `noop` packet it can...

Hmm it could never be as reliable or as efficient as checking just before sending the message. Set interval is also paused so the interval would need to be quite...

This isn't supported out of the box but it should be possible to achieve with the api Could also potentially add an api method to get an array of all...

So you want them to be constantly looping until the button is clicked and then stop after the button is clicked when a specific item is on screen? Not stop...

Hey @bencius where are you running this? It would be great if you could provide an example on stackblitz with the code you're running.

Ok. Can you reproduce it if you run the same code in a browser? If you could share the js you're running that would help debug. I'm on mac and...