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

Slot machines directly use bank account balance #5193

Merged
merged 9 commits into from
Jul 10, 2021

Conversation

112358sam
Copy link
Contributor

@112358sam 112358sam commented Jun 18, 2021

[QoL] [input-wanted]

About the PR

Slot machines now directly use a player's bank account similar to vending machines. The withdraw to card and deposit from card functions are removed from the ATM as they were only used for slot machines. To maintain account security, slot machines now ask for a PIN before allowing play.

The slot machine UI now has the player cash in some amount from their account before play and cash out after play. Removing the card also cashes out all funds. This prevents someone with bank access from pulling someone's winnings while they are still playing.

image

Why's this needed?

Having two digital currencies is unintuitive and difficult for new players to learn. The withdraw to card feature is only used for slots and seems out of place with every other device using bank account funds.

Changelog

(u)112358sam
(+)Slot machines use bank account funds.
(+)ATMs no longer withdraw to and deposit from ID cards.

@keywordlabeler keywordlabeler bot added the C-QoL A quality of life improvement that makes the game easier to play label Jun 18, 2021
@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jun 18, 2021
@112358sam 112358sam changed the title slots-use-bank Slot machines directly use bank account balance Jun 18, 2021
@RGBDeadSilent
Copy link

This does make it easier on new players, but it also makes it easier for antags to immediately embezzle the money directly out of your account.

HONK HONK HONK! JUSTIN FALKLIN JUST HIT IT BIG AND WON 1,000,000 CREDITS!

Let me just immediately steal that as a HoP antag, move it to somewhere I can easily pull it from, and you've now lost that money immediately.

At least the old way it was on the card, and the antag had a chance to move it to their account when they hit listening post to purchase things. (What other reasons are there to win big? Buying Station destroying pod weapons and AEX ammo thats what.)

@112358sam
Copy link
Contributor Author

112358sam commented Jun 18, 2021

Didn't expect this to be contentious. I'll add the input-wanted label. I'll say that is a good point though. I couldn't think of any downsides while I was coding it. The counter could be that it would pretty quickly out the HoP as antag and the player can still withdraw cash to keep it safe. Not perfect though. I am interested to see where the consensus ends up.

@Studenterhue Studenterhue added E-Add-To-Wiki A PR that will require changes to the wiki E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. labels Jun 20, 2021
@github-actions
Copy link
Contributor

Created discussion thread: https://forum.ss13.co/showthread.php?tid=16647

Copy link
Contributor

@MarkNais MarkNais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As much as instant-embezzlement is a possibility, I personally don't find it to be a blocker.

code/obj/submachine/slots.dm Outdated Show resolved Hide resolved
@boring-cyborg boring-cyborg bot added the A-UI Modifies UI in some way. Automatically applied on a change to tgui/ label Jul 4, 2021
@112358sam
Copy link
Contributor Author

112358sam commented Jul 4, 2021

I've changed the slot machine to locally hold play funds until the card is ejected. I've edited the description accordingly. This should help deal with the embezzlement possibility.

Copy link
Contributor

@MarkNais MarkNais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stored Funds is a neat idea. I assume you tested the changes.

@112358sam
Copy link
Contributor Author

Stored Funds is a neat idea. I assume you tested the changes.

Yeah, I loaded it up and tested a variety of cases to make sure everything worked as intended.

@ZeWaka ZeWaka merged commit 44910e6 into goonstation:master Jul 10, 2021
github-actions bot pushed a commit that referenced this pull request Jul 10, 2021
@112358sam 112358sam deleted the slots-use-bank branch July 10, 2021 18:33
@Studenterhue Studenterhue removed the E-Add-To-Wiki A PR that will require changes to the wiki label Jul 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-UI Modifies UI in some way. Automatically applied on a change to tgui/ C-QoL A quality of life improvement that makes the game easier to play E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants