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

[MIRROR] Fix bug where MMI's cannot open doors #905

Merged
merged 1 commit into from
Dec 2, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25367
Original PR: tgstation/tgstation#79896

About The Pull Request

While #69556 did fix it for posi brains. it did not account for MMI's meaning only posi brains got fixed

as a bonus, im throwing in a shitty debug command because MMI's put all there logic in attackby

Why It's Good For The Game

bugs bad

Changelog

🆑 Autisem
admin: A new debug verb to turn yourself into an MMI(almost the funniest thing)
fix: MMI's inside mechs can now properly open doors like there posibrain counterparts
/:cl:

* Fix bug where MMI's cannot open doors (#79896)

## About The Pull Request

While #69556 did fix it for posi brains. it did not account for MMI's
meaning only posi brains got fixed

as a bonus, im throwing in a shitty debug command because MMI's put all
there logic in attackby

## Why It's Good For The Game

bugs bad

## Changelog
:cl:
admin: A new debug verb to turn yourself into an MMI(almost the funniest
thing)
fix: MMI's inside mechs can now properly open doors like there posibrain
counterparts
/:cl:

---------

Co-authored-by: san7890 <the@ san7890.com>

* Fix bug where MMI's cannot open doors

---------

Co-authored-by: Autisem <[email protected]>
Co-authored-by: san7890 <the@ san7890.com>
@ReezeBL ReezeBL merged commit 441d0ef into master Dec 2, 2023
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-25367 branch December 2, 2023 15:19
AnywayFarus added a commit that referenced this pull request Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants