Closed Sean-Der closed 6 months ago
Thanks for the review @biglittlebigben I will fix that up now!
Here is a jsfiddle of how I was testing https://jsfiddle.net/phn51xz9/
Ok I feel pretty good about that @biglittlebigben
Testing with clients should happen Tuesday/Wednesday!
Please let me know when you happy with your testing and this is ready to merge.
@biglittlebigben testing went well, this is good for merge. I had to make one additional change though.
We now close the session when goes to failed (30 seconds of no activity) instead of disconnected (5 seconds of no activity)
The client side is waiting for 'disconnected'.
Marking as draft. Should be tested against a client before worth reviewing