Jacob-Griffin / TelephonePictionary2.0

New version of blowyourfaceoff.com using more modern web technology than php and raw javascript
0 stars 0 forks source link

Round Not Moving options #9

Closed Jacob-Griffin closed 1 year ago

Jacob-Griffin commented 1 year ago

I don't estimate that this will be nearly as big a problem with this version of the game since the player will not start their waiting unless the submit was successful, but there should be a few options if a player disconnects or somehow misses their send and can't get back into the round.

Jacob-Griffin commented 1 year ago

People can rejoin games if their status is 'missing', and they put in their name correctly. Firebase can take a simple handle for onDisconnect, which includes any navigation away from the gameplay page, based on a reference the client passes them (ex. "Here's where my player status is, if I leave for any reason, mark me missing").

The lingering thing that might need some doing is getting your username exactly right on rejoin, but we can include a button specifically for rejoining a broken game, based on info I've stored locally. Edit: I went with making it autofill username and gameid if backed up game data exists

Jacob-Griffin commented 1 year ago

Thinking through this one again, now that rejoining an in progress game is possible

I'll only change the checkboxes on the original post for posterity, and put my new expectations for the remaining tasks afterwards

Jacob-Griffin commented 1 year ago

New "host options" for if a game is stuck

These should appear for the host if the time goes over by several seconds without the round moving on

Jacob-Griffin commented 1 year ago

Short of a player that:

round safety options should be pretty robust. Most disconnects will be for players that'll be right back in, and between adding time and rejoin game options, there should be no problem accommodating a rejoining player. With this process being noted in the "Knowlege Base", and said kb being linked to when the round is "stuck", the core of the issue is pretty much gone.

With the edge case above, it may be worth adding "Remove missing players" as a very low priority option for the future, in its own issue, but seems like mostly reasonable workarounds for now.

That being said, moving that out into its own issue (#26), the rest of this is complete. Closing