Closed ahv15 closed 4 years ago
Also, it would be cool to have a timer which would indicate when the button will be active! But that is deffinitelly for a next PR.
This is very weird the restart button should not be visible for 10 seconds even for the host
This is very weird the restart button should not be visible for 10 seconds even for the host
huh 🤔 I did had it right away for the host. Was there maybe any other step to test this other than joining with local clients to a game?
It works fine when I run it from my workspace This is how it looks for me just checked it out again.
Note: there is a hasRestartPermission
Boolean variable at GameOverEvent.java
, could you consider removing it as well if it is not needed elsewhere?
Can confirm the tweak from @stefaniamak did the final trick and collapsed the quantum code state into one that works all the time, not just all the time for some and none of the time for others 😁
The last few bits of the restart permission were trivial to remove so I did it real quick and added it to a commit here. 5 total commits with 4 contributors, now that's team work. Well done everybody and thank you! Realistically replayable LaS in headless multiplayer FTW!
Contains
"Fixes #154" Allows all the players to restart the game. The restart button appears after 10 seconds allowing the players to check their stats in the mean time.
How to test
Host and Join a multiplayer game using 2 or more host/clients and once the game is completed everybody should be able to restart the game.
Outstanding before merging