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

Enforce active turn for first warring human #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

bcalebrobert
Copy link

Who knows if this will have bad side-effects, but it seems to
work in the lobby.

Test plan:

Load up my bad save game and check that India's turn is active
in the lobby. It does. Hopefully we can make it through India's
turn and then through every other warring players turns too.
Fingers crossed on that.

Also, I tried to reproduce the bad save game in a two player
game. Unfortunately (?), this game seems to load fine. I can
load this working save with the patch applied, and have verified
that it doesn't break anything.

Who knows if this will have bad side-effects, but it seems to
work in the lobby.

Test plan:

Load up my bad save game and check that India's turn is active
in the lobby. It does. Hopefully we can make it through India's
turn and then through every other warring players turns too.
Fingers crossed on that.

Also, I tried to reproduce the bad save game in a two player
game. Unfortunately (?), this game seems to load fine. I can
load this working save with the patch applied, and have verified
that it doesn't break anything.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants