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

[1856] must sell in blocks when forced to sell #11312

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

Conversation

philcampeau
Copy link
Collaborator

Fixes #11311

Before clicking "Create"

  • Branch is derived from the latest master
  • Add the pins or archive_alpha_games label if this change will break existing games
  • Code passes linter with docker compose exec rack rubocop -a
  • Tests pass cleanly with docker compose exec rack rake

Implementation Notes

Explanation of Change

in 1856 the rules state that forced sales must be done in blocks. While we could write more complicated code that only enforces this during forced sales, there's really zero reason anyone would not sell in blocks during an SR, since share price drops by share sold, not by block sold.

I've set MUST_SELL_IN_BLOCKS to true for the game, which would also impact 1836jr56. As a result, I had to remove one fixture game for 36jr56 which failed Rake.

Let me know if you disagree with this approach. Some games may need pinning.

Screenshots

Any Assumptions / Hacks

@philcampeau philcampeau added pins PR that will require some games to be pinned 1856 labels Nov 1, 2024
@ollybh
Copy link
Collaborator

ollybh commented Nov 11, 2024

Did the rules interpretation about selling in blocks get confirmed? I recall seeing some messages about this on the 18xx mailing list.

@philcampeau
Copy link
Collaborator Author

The rule wasn't clarified on the list, but like I pointed both there and above, there really is no functional reason why you wouldn't sell in blocks during the Stock Round in a game without price protection.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1856 pins PR that will require some games to be pinned
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1856 Id: 183019 - allows shares to be sold individually when doing forced stock sales
2 participants