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

The Great Shiptest IRS Audit #3915

Closed
wants to merge 39 commits into from
Closed

Conversation

Geoengi
Copy link
Contributor

@Geoengi Geoengi commented Dec 17, 2024

About The Pull Request

  • Removes the ability to place material coins into slot machines by changing the pay mode; they only work with cash bundles now. Thank @flopppi for this wonderful edit.
  • The roulette table no longer gets stuck with spinning numbers and now references bank cards over ID cards.
  • Vendors no longer ask for ID cards when no account is presented and instead bank cards.

Why It's Good For The Game

So turns out that if you place coins in a slot machine and go to withdraw them or deconstruct the machine, it literally tries to throw infinite coins at your head and kills the server. Obviously, it should not do this.

Changelog

🆑
fix: Partially fixed persistent overlay on roulette tables.
tweak: Replaced references to ID cards with bank cards where applicable.
code: Slot machines no longer accept material coins due to them nuking the server if withdrawn.
/:cl:

@github-actions github-actions bot added TGUI Code change Watch something violently break. labels Dec 17, 2024
@Ratvarr
Copy link

Ratvarr commented Dec 17, 2024

I think you messed up your commits or. something. your merging your master not branch

@Geoengi
Copy link
Contributor Author

Geoengi commented Dec 17, 2024

woops - will use branch

@Geoengi Geoengi closed this Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break. TGUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants