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] Fixes SMES terminal placing under the SMES and not under the player #2908

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#2003
Original PR: tgstation/tgstation#82665

About The Pull Request

Changes src touser to get intended behavior.

image

image

Why It's Good For The Game

if only we had a way to place both input and output on a single turf....

Changelog

🆑 Iajret
fix: wiring newly built SMESes should place new teminal under the player and not under the SMES itself.
/:cl:

… player (#2003)

* Fixes SMES terminal placing under the SMES and not under the player (#82665)

## About The Pull Request

Changes `src` to`user` to get intended behavior.

* Fixes SMES terminal placing under the SMES and not under the player

---------

Co-authored-by: Iajret <[email protected]>
@Iajret Iajret merged commit 7e3905c into master Apr 16, 2024
26 of 27 checks passed
@Iajret Iajret deleted the upstream-mirror-2003 branch April 16, 2024 09:13
AnywayFarus added a commit that referenced this pull request Apr 16, 2024
Iajret pushed a commit that referenced this pull request Jun 8, 2024
…IGNORE] (#2908)

* Fix admin borg panel unable to install/remove upgrades (#83733)

## About The Pull Request
Fixes the admin borg panel's upgrade functionality. Caused by not
registering the signals for deletion and also by the upgrade code
checking the robot's contents instead of upgrades list (since the borg
panel spawns the item in the borg, it appears in the borg's contents,
making the check think the borg already has it installed.)
## Why It's Good For The Game
Easier testing new borg modules
## Changelog
:cl:
fix: Fixes admin borg panel upgrade functions
/:cl:

* Fix admin borg panel unable to install/remove upgrades

---------

Co-authored-by: thegrb93 <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
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