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] Surgery fixes #1483

Merged
merged 1 commit into from
Nov 23, 2023
Merged

[MIRROR] Surgery fixes #1483

merged 1 commit into from
Nov 23, 2023

Conversation

SierraHelper
Copy link
Collaborator

Оригинальный PR: Baystation12#34311

🆑 emmanuelbassil
bugfix: Can now use welders to fix external dents on IPCs on an operating table
bugfix: Fixed oversight where you could not use swappers for surgical steps.
bugfix: Fixed oversight where welding tool lit up and damage your eyes even if no actual surgical step occured.
/🆑

So keep in mind, swappers in existing behavior don't actually check which head is equipped before allowing the surgery. Given I am not that comfortable with surgery code yet, I did not fix it in the scope of this PR. Mechanically it's not a big problem if the swapper doesn't actually check since it's a click away anyways.

I also sadly had to reintroduce the 'check if surgeries are possible' bit to allow welding tools to work on surgical tables.
The rest of the 'welders harm your eyes for no reason' oversight is fixed in my other open PR.

@SierraHelper SierraHelper requested a review from a team as a code owner November 20, 2023 12:17
@SierraHelper SierraHelper added 🪞 MIR ЯОЯ 📜 CL валиден Ченджлог не содержит ошибок labels Nov 20, 2023
@SuhEugene SuhEugene merged commit c70eecd into dev-sierra Nov 23, 2023
13 checks passed
@SuhEugene SuhEugene deleted the upstream-pr-34311 branch November 23, 2023 09:31
SierraHelper added a commit that referenced this pull request Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪞 MIR ЯОЯ 📜 CL валиден Ченджлог не содержит ошибок
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants