Skip to content
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

Show joined players in waiting room #233

Open
JoshWein opened this issue Mar 27, 2021 · 3 comments
Open

Show joined players in waiting room #233

JoshWein opened this issue Mar 27, 2021 · 3 comments
Labels
bug A bug.

Comments

@JoshWein
Copy link

Describe the bug
It should show players currently in the waiting room so it's easier to tell if all players have joined

Browser:

  • All
@JoshWein JoshWein added the bug A bug. label Mar 27, 2021
@JoshWein
Copy link
Author

This seems like it should show up, I do see it when I test locally but not on the live version;

https://github.com/amritrau/shadow-hunters/blob/master/shadow-hunters/app.py#L452

@amritrau
Copy link
Owner

Hi @JoshWein -- thanks for flagging this! Hope you're enjoying the game otherwise. This is a strange one: I see it consistently when testing locally, but the "Other players in the room:" message only appears sometimes on the deployed site.

Given that we've previously seen issues around clients dropping and reconnecting, my initial suspicion that this reconnect guard might be the problem (cc @mlevatich, would love to hear your thoughts as well)

if not reconnect:
socket_tell(msg, [], None, room_id, client=(request.sid,))

@mlevatich
Copy link
Collaborator

It's not immediately apparent to me what's happening here, but I agree with your suspicion @amritrau – I should have some time in the next couple of days to look into it more and figure out what's going on.

amritrau added a commit that referenced this issue Apr 8, 2021
mlevatich pushed a commit that referenced this issue Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug.
Projects
None yet
Development

No branches or pull requests

3 participants