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

Re-entries and lammers #1496

Open
ctm opened this issue Sep 11, 2024 · 1 comment
Open

Re-entries and lammers #1496

ctm opened this issue Sep 11, 2024 · 1 comment
Assignees
Labels
bug Something isn't working easy Trivial to do (even when tired!) and semi-worthwhile high priority Should be done fairly soon

Comments

@ctm
Copy link
Owner

ctm commented Sep 11, 2024

Disallow the setting of the last re-entry level beyond the last lammer level.

Lammers are relatively rare, but they were actually used in the 2022 and 2023 WSOP short deck tournaments, both of which exist as structures in mb2. In both of those structures, both the last lammer level and the last re-entry level is level eight. Currently, the tournament creation UI allows people to change the last re-entry level and we typically set the last re-entry level to level nine for evening tournaments. That creates a problem that hasn't been hit (AFAIK), in that players can re-enter after lammer redemption.

I haven't even tested to see what happens if a player re-enters after redemption, but my guess is the player then gets two lammers which might not even be redeemable.

Lammer info is not currently configurable at the database level; it exists solely in the structures, which are currently hardcoded. So, although there are alternative solutions to the problem of having a re-entry after the mandatory lammer redemption, simply disallowing them at event creation is the easiest and is what should be done now.

@ctm ctm added bug Something isn't working high priority Should be done fairly soon easy Trivial to do (even when tired!) and semi-worthwhile labels Sep 11, 2024
@ctm ctm self-assigned this Sep 11, 2024
@ctm
Copy link
Owner Author

ctm commented Sep 11, 2024

I've done the work on the client side and will be deploying soon. I will add something on the server side before closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working easy Trivial to do (even when tired!) and semi-worthwhile high priority Should be done fairly soon
Projects
None yet
Development

No branches or pull requests

1 participant