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

1844 Handling of SBB shares. ID: 137304 #9792

Closed
Lime-Turtle opened this issue Oct 17, 2023 · 3 comments · Fixed by #9822
Closed

1844 Handling of SBB shares. ID: 137304 #9792

Lime-Turtle opened this issue Oct 17, 2023 · 3 comments · Fixed by #9822
Labels
1844 share size Issues relating to unclear share sizes

Comments

@Lime-Turtle
Copy link

(https://18xx.games/game/137304)

It seems that 5% shares are valued at 50% of SMP, but should be full (and 10% share should be twice SMP, but are treated as SMP).

Player portfolios do not clearly indicate share holding split (5% vs 10%)

In this game I had 25% and 210% shares. In an SR I was given the opportunity to sell 5, 10, 20 or 30%. But I was not allowed to sell 15% or 25%. (I didn't intend to sell any, I was merely checking the implementation)

@philcampeau philcampeau added share size Issues relating to unclear share sizes 1844 and removed needs triage labels Oct 18, 2023
@philcampeau
Copy link
Collaborator

It seems that 5% shares are valued at 50% of SMP, but should be full (and 10% share should be twice SMP, but are treated as SMP).

This is correct, and seems to be a mistake in the code.

Player portfolios do not clearly indicate share holding split (5% vs 10%)

Perhaps a better approach would be to treat 5% shares as 1 cert, and the 10% shares as double certs, like the ones in 1849, 1847 AE, and a couple of others.

@crericha
Copy link
Collaborator

crericha commented Oct 18, 2023

I'm open to double certs, but I don't understand how that will help if you have two of them? How will you know? For example, if I have 10%, 5%, 5% or 10%, 10%...won't both show up as d 4?

@crericha
Copy link
Collaborator

Partially fixed by #9807. The shares have the correct value now, but the sell share interface is still weird.

@crericha crericha linked a pull request Oct 24, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1844 share size Issues relating to unclear share sizes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants