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

Fixes some chat messages with the arcade prize machine #23354

Merged
merged 3 commits into from
Nov 29, 2023

Conversation

BrickTheHooman
Copy link
Contributor

What Does This PR Do

Fixes chat messages meant just for the user going to everyone within view.

Why It's Good For The Game

Generally makes more sense, not everyone needs to see that you're too arcade poor to afford the water balloon

Testing

Fired up the server with 2 clients, made sure only the machine user could see it when the purchase went through and when it didn't

Changelog

🆑
fix: Arcade reward machine no longer broadcasts your lack of tickets to the world
/:cl:

@ParadiseSS13-Bot
Copy link
Contributor

Your PR has been put on hold as your GitHub account is not linked to a Paradise forums account.

Please link your account at https://www.paradisestation.org/forum/custom/link_github_account.php

Headcoders may ignore this in specific circumstances (Minor fix PR, trusted contributor from another codebase, anything at their discretion)

@ParadiseSS13-Bot ParadiseSS13-Bot added On Hold Its gonna be a while before this is reviewed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally labels Nov 24, 2023
Copy link
Member

@Burzah Burzah left a comment

Choose a reason for hiding this comment

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

While we are here.

code/modules/arcade/prize_datums.dm Outdated Show resolved Hide resolved
code/modules/arcade/prize_datums.dm Outdated Show resolved Hide resolved
@farie82 farie82 added Fix This PR will fix an issue in the game and removed On Hold Its gonna be a while before this is reviewed labels Nov 29, 2023
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting merge This PR is ready for merge and removed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally labels Nov 29, 2023
@lewcc lewcc merged commit a6697de into ParadiseSS13:master Nov 29, 2023
Oyu07 pushed a commit to Oyu07/Paradise that referenced this pull request Jan 2, 2024
…23354)

* fixes the chat message

* addresses review

* addressing review 2 electric boogaloo
@BrickTheHooman BrickTheHooman deleted the the-first-attempt branch March 24, 2024 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting merge This PR is ready for merge Fix This PR will fix an issue in the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants