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

Merge maintenance branch into the master branch #170

Draft
wants to merge 59 commits into
base: master
Choose a base branch
from

Conversation

madoar
Copy link
Contributor

@madoar madoar commented Apr 18, 2020

This PR merges the maintenance branch into the master branch.

While doing this I tried to change the master branch as little as possible to make this PR readable.

neutronc and others added 30 commits October 6, 2018 17:39
with only 10 shares, the last 10 shares get deleted even though they
might be owned by a player. This stems from the fact that
findcertificate() returns the certificates within rails 2.0 in a
different order than previous releases.
with only 10 shares, the last 10 shares get deleted even though they
might be owned by a player. This stems from the fact that
findcertificate() returns the certificates within rails 2.0 in a
different order than previous releases.
company if the route was blocked by a token of another major company.
This has been corrected
Implementing the mapHex block for private companies and the phase 4 tile
blocks for upper italy.
Fixing Map coordinates
Implementing Change to Stockmarket
Dont Merge Minors if the Corresponding Major has been sold out.
Final Oversight for K2 in Startpackage..
Fixing Hungarian Private Certificates Count
Fixing Hungarian U1 and U3 starting cash
Company Treasury
Incomes.

This is needed for a seperate income during the OR not tied to the
private company income but stemming from the additional revenue coming
from special trains (Coal Trains in 1837) or special powers like a Mail
Contract.
Company Treasury
Incomes.

This is needed for a seperate income during the OR not tied to the
private company income but stemming from the additional revenue coming
from special trains (Coal Trains in 1837) or special powers like a Mail
Contract.
first work on revenue recognition for Coal Mines
neutronc and others added 29 commits January 4, 2019 11:15
https://github.com/neutronc/Rails-1.git into rails_2_maintenance

Conflicts:
	src/main/resources/data/1837/Game.xml
the Startpackets to not be choosen.

Whitespace in Variant Options are dangerous
they have been upgraded already.
Might need some additional work in the future if theres the rare case a
company doesnt touch its reserved hex.
Preparation of Step 2 OR Round Checks
Adding new mechanism to direct correct handling of OR and SR fter
CoalexchangeRounds
Previously it wasnt possible to dump x shares containing the presidency.
Now it is. Befoire this fix you had to sell twice shares from the
company with the president in your hand. Now the correct amount is
available on the first sell.

Bug reported by Peter De Boer, December 2019
…llowed to split sell shares.."

This reverts commit b392b6d.
testcase. All other testcases report clear handling sofar.
emergency buy and wasnt declared bancrupt.

Now the player is declared bancrupt and the game endswith bancruptcy if
it should end.
Fixing Bug reported by Peter De Boer: Player cant sell any shares in
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.

3 participants