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

Add new missed turn schema #9

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Conversation

jmo1121109
Copy link
Owner

PR to see merge conflicts

jmo1121109 and others added 17 commits February 10, 2019 21:38
…ames, wD_Users, wD_Members and increased webdip version according to DB value
- CD takeovers get excuses (TODO: Handling of takeovers from members who left before)
- Adjusted old NMR code to count missed phases even before adjudication (if an NMR happened, the game will not be progressed so there is no need to do it after the processing in case of a crash) and to reset it if no NMR happened; removed code to reset the count when the member returns to the game (the mere purpose of this count is now to notify other  members about the NMR)
- Small fixes to get the last commit actually running
Members with no complete set of orders lose an excuse.

If no excuses are left, members go into CD.

The phase only progesses if there are orders from each country, that has to order.

Members are notified about phase extensions and use up of excuses.

TODO: Further handling of CDs that goes beyond the current state (temp ban etc).
Code can be directly added to Member->setLeft
Basic implementation of new NMR schema
…ble for system temp bans (for new NMR schema)
	- A left position does not hold up the game.
	- Left positions are now handled correctly.
	- Temp Ban sanctions are now added for missed turns without excuses.

Methods for updating MissedTurnCounts during NMR recording added. Though I am not completely sure what data should be stored there so I left them commented.
jmo1121109 pushed a commit that referenced this pull request May 26, 2019
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