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

Check for retreat options before win #322

Merged
merged 1 commit into from May 21, 2018

Conversation

jmo1121109
Copy link
Contributor

Previously if a player got 18 centers in the fall the game would declare
them the winner before the retreat phase. Now if there is a retreat
phase, the game will wait and check for the winner after retreats are
done.

Previously if a player got 18 centers in the fall the game would declare
them the winner before the retreat phase. Now if there is a retreat
phase, the game will wait and check for the winner after retreats are
done.
@jmo1121109
Copy link
Contributor Author

This has been live on vDip for some time now without causing any problems. Local testing was also done, but I will not push this live until I have a game live on the server in the fall season with 17 centers and a retreat into an open center available for testing. If the live test does not work for any reason I'll be rolling back this change immediately due to the wide scope of games it could impact.

@jmo1121109
Copy link
Contributor Author

Will correct #196

@TimothyJones TimothyJones merged commit 2b93e3b into kestasjk:master May 21, 2018
@jmo1121109 jmo1121109 deleted the Fix-Win-Before-Retreats branch May 28, 2018 18:12
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.

None yet

2 participants