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

[Port from /tg/] Fixes Cell Message With Mulebots #779

Merged

Conversation

LikeLakers2
Copy link
Contributor

(The following is pulled from tgstation/tgstation#77374, which this is a port of.)

About The Pull Request

Fixes the message when removing/adding a cell to mulebots using the mulebot as the user, resulting in sentient mulebots receiving the message "You add/pry out the cell in the mulebot" despite not being the one adding/removing the cell. First time making a pr for this codebase, so let me know if I fucked something up.

Why It's Good For The Game

The mulebot isn't moving the cell, whoever IS moving it is. Also bug bad.

Changelog

🆑 Wisemonster
fix: Fix mulebots receiving the wrong message when thier cell is added/removed.
/:cl:

## About The Pull Request
Fixes the message when removing/adding a cell to mulebots using the
mulebot as the user, resulting in sentient mulebots receiving the
message "You add/pry out the cell in the mulebot" despite not being the
one adding/removing the cell. First time making a pr for this codebase,
so let me know if I fucked something up.
## Why It's Good For The Game
The mulebot isn't moving the cell, whoever IS moving it is. Also bug
bad.
## Changelog
:cl:
fix: Fix mulebots receiving the wrong message when thier cell is
added/removed.
/:cl:

---------

Co-authored-by: san7890 <[email protected]>
@wraith-54321 wraith-54321 merged commit b387bd9 into Monkestation:master Dec 23, 2023
15 of 21 checks passed
github-actions bot added a commit that referenced this pull request Dec 23, 2023
@LikeLakers2 LikeLakers2 deleted the fix-mulebot-prying-messages branch December 23, 2023 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants