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 that one mining vendor runtime #10841

Merged
merged 2 commits into from
Apr 5, 2024

Conversation

JixS4v
Copy link
Contributor

@JixS4v JixS4v commented Apr 4, 2024

About The Pull Request

Vendors no longer runtime because they tried to place a mob into your hand.
Fixes #10272

Why It's Good For The Game

Runtimes bad

Testing Photographs and Procedure

Screenshots&Videos
testing.evidence.mp4

Changelog

🆑 Ratón
fix: Vendors will no longer try to put mobs in your hands
/:cl:

code/modules/mining/machine_vending.dm Outdated Show resolved Hide resolved
@PowerfulBacon PowerfulBacon added this pull request to the merge queue Apr 5, 2024
Merged via the queue into BeeStation:master with commit 2199ee9 Apr 5, 2024
8 checks passed
DrDuckedGoose pushed a commit to DrDuckedGoose/BeeStation-Hornet that referenced this pull request May 11, 2024
* The thing

* Update code/modules/mining/machine_vending.dm

Co-authored-by: Tsar-Salat <[email protected]>

---------

Co-authored-by: Tsar-Salat <[email protected]>
DrDuckedGoose pushed a commit to DrDuckedGoose/BeeStation-Hornet that referenced this pull request May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: /obj/machinery/vendor/mining non-item runtime bug
4 participants