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 smart pipe & cryo gas reallocation #2954

Merged
merged 1 commit into from
Apr 19, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#2070
Original PR: tgstation/tgstation#82701

About The Pull Request

1. Smart Pipe Problem

  • Fill a pipe with any gas, plasma for e.g.
  • Unwrench the pipe to deconstruct
  • The pressure sends you flying but notice plasma is nowhere to be seen from the now destroyed pipe

2. Cryo pipe connector problem
The gas gets reallocated when the object is deleted. We only want this when its deconstructed not deleted to prevent any side effects

This PR fixes them

Changelog

🆑 SyncIt21
fix: smart pipes release their gases into the air when unwrenched
fix: cryo pipe connector component has no side effects of gas reallocation when deleted
/:cl:

* Fixes smart pipe & cryo gas reallocation (#82701)

## About The Pull Request

**1. Smart Pipe Problem**
 - Fill a pipe with any gas, plasma for e.g.
 - Unwrench the pipe to deconstruct
- The pressure sends you flying but notice plasma is nowhere to be seen
from the now destroyed pipe
 
**2. Cryo pipe connector problem**
The gas gets reallocated when the object is deleted. We only want this
when its deconstructed not deleted to prevent any side effects

This PR fixes them

## Changelog
:cl:
fix: smart pipes release their gases into the air when unwrenched
fix: cryo pipe connector component has no side effects of gas
reallocation when deleted
/:cl:

* Fixes smart pipe & cryo gas reallocation

---------

Co-authored-by: SyncIt21 <[email protected]>
@ReezeBL ReezeBL merged commit b4069e5 into master Apr 19, 2024
27 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-2070 branch April 19, 2024 07:18
AnywayFarus added a commit that referenced this pull request Apr 19, 2024
Iajret pushed a commit that referenced this pull request Jun 10, 2024
* ERT_Bounty.DMM ship touchup (#83608)

## About The Pull Request

It updates this shuttle to make it better!
I'll be honest, I don't _actually_ know what this is used for. Maybe an
admin button makes this spawn? I'm not sure. Eitherway, it's better now.

## Why It's Good For The Game

This place looks better; mapping improvement.

## Changelog
:cl:
qol: The ERT's ship has better mapping now.
/:cl:

* ERT_Bounty.DMM ship touchup

---------

Co-authored-by: Fazzie <[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