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

Lotto Vendor Mapping #32121

Merged
merged 1 commit into from
Feb 28, 2022
Merged

Conversation

west3436
Copy link
Collaborator

@west3436 west3436 commented Feb 22, 2022

Adds lotto vendors to each station's Bar.

Ideally this should be merged after #32120 so people don't exploit hacking hack the machines for free money.

[general]
🆑

  • rscadd: lotto vendors can now be found in each station's Bar.

@PJBot PJBot added the Mapping (General 🗺️) This applies to every map really. label Feb 22, 2022
@Eneocho
Copy link
Collaborator

Eneocho commented Feb 22, 2022

Hacking for free items is not an exploit and that function is there for the sole purpose of dispensing free items.

@west3436
Copy link
Collaborator Author

Hacking for free items is not an exploit and that function is there for the sole purpose of dispensing free items.

Wrong PR. This just places the vendors on each map.

@Eneocho
Copy link
Collaborator

Eneocho commented Feb 23, 2022

I know, but you literally said

so people don't exploit hacking.

on the OP.

@west3436
Copy link
Collaborator Author

I know, but you literally said

so people don't exploit hacking.

on the OP.

Please read that entire sentence in the OP. It is just implying you shouldn't merge this early otherwise an exploit may be available. Your comments don't address anything regarding the PR itself, which only maps vendors in.

@Kurfursten Kurfursten added the Emoji This 👍 👎 Nobody's given their opinion yet label Feb 24, 2022
@DamianX DamianX merged commit 9b31705 into vgstation-coders:Bleeding-Edge Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Emoji This 👍 👎 Nobody's given their opinion yet Mapping (General 🗺️) This applies to every map really.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants