crybapp / api

Core service used to handle events from clients
MIT License
36 stars 8 forks source link

Changes to enable queue status #86

Closed Soliel closed 4 years ago

Soliel commented 4 years ago

Having a second though, something came on mind... The queue status is only sent to all of the clients inside the room at the moment, but if someone reconnects/joins later while still in queue, they wouldn't get the queue position. Storing this somewhere temporarily where it can be accessed, and then sending that on connection would work.

This gets rectified when a QueueMovement event is sent, which happens every 15 seconds. so the most time they'll be waiting to see the queue is 15 seconds, I personally think this is better than sending a request to portal asking to update the queue status for this user.

JeDaYoshi commented 4 years ago

Then we can continue, then.