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

Game can't end unless a human wins [GH-2] #11

Closed
ben-ryder opened this issue Feb 23, 2020 · 3 comments
Closed

Game can't end unless a human wins [GH-2] #11

ben-ryder opened this issue Feb 23, 2020 · 3 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@ben-ryder
Copy link
Owner

Describe the bug
If all human players die, the game does not end and the last surviving player must keep clicking through the next turn message as the computer players continue.
If a computer player wins, the game will not end but will continue. This is because the check for player death is done in the conquer handling in the view which the computer player doesn't invoke.

To Reproduce
All human players must be conquered, and a computer player must then conquer all other players.

Expected behaviour
When a computer player wins, the game should end and there should be the appropriate message.
When all human players die but computer players remain, this could be handled in multiple ways.

  • The game could end with a defeated message.
  • The next turn message continues each turn cycle but the text changes to reflect that they are simply viewing the game unfold now.
  • The last player has a choice to keep watching the game by clicking through each turn cycle or to end the game.

System Information (please complete the following information):
n/a (This is a code bug/issue)

Additional context
n/a

@ben-ryder ben-ryder added the bug Something isn't working label Feb 23, 2020
@ben-ryder ben-ryder added this to the v2.0 milestone Feb 23, 2020
@ben-ryder ben-ryder self-assigned this Feb 23, 2020
ben-ryder added a commit that referenced this issue Jun 13, 2020
@ben-ryder
Copy link
Owner Author

I've now created a game save which can be used to reproduce this issue. See issue 11 reproduction.zip.

It looks like the exact issue behavior has changed since this issue was first raised:

and the last surviving player must keep clicking through the next turn message as the computer players continue.

This is no longer accurate. What happens now is that the next turn message for the last surviving human stays on the screen and the game freezes. The backend then runs though the rest of the game (computers only) in the background. When the game is over and there is one surviving computer player, the UI updates again and the game over message is displayed.

@ben-ryder
Copy link
Owner Author

I'm going to look to fix this by implementing option one where the game will end with a different game over message.
This will probably work similar to the current game over message handling, but will instead be triggered when there are no human players left.

ben-ryder added a commit that referenced this issue Sep 12, 2021
@ben-ryder
Copy link
Owner Author

This should now be fixed and this issue should no longer block work on GH-2.

COE - Humans Defeated Message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant