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

Logging for Inserting & Removing Things Surgically #9873

Merged
merged 2 commits into from
Sep 22, 2023

Conversation

VickiMorris
Copy link
Contributor

@VickiMorris VickiMorris commented Sep 21, 2023

About The Pull Request

Adds logging for additional surgical processes, as currently with the three operations affected in this PR, it logs when each is started and when an organ is removed via Organ Manipulation. This makes it so organ manipulation logs when organs are inserted as well, cavity implant logs when items are put into or taken out of a body cavity surgically, and when implants are surgically removed from someone or they are checked for implants.

Adds logging for when any of the above procedures are attempted as well, encase the surgeon is interrupted mid procedure.

Why It's Good For The Game

Requested by a Maintainer, seconded by a Seniormin, more logging is good.
Closes issue #9843

Testing Photographs and Procedure

Screenshots&Videos

Organ Manipulation 2
Cavity Implant 2
Implant Removal 2

Changelog

🆑 Impish_Delights
admin: additional logging for organ manipulation, cavity implant, and implant removal surgeries
/:cl:

Adds logging for inserting new organs via Organ Manipulation.
Adds logging for extracting implants from people.
Adds logging for storing or removing items surgically using a cavity implant.
Copy link
Member

@EvilDragonfiend EvilDragonfiend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would please you add logging when they 'attempt' an operation aside finished operation?

Adds an additional log for when someone attempts to implant/remove organs, implant/remove cavity items, and remove/check for implants.
Removed user intent for surgical steps, as they can only be done on help, or disarm (Intentional Failure) which causes a combat log anyway.
Copy link
Member

@EvilDragonfiend EvilDragonfiend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@EvilDragonfiend
Copy link
Member

EvilDragonfiend commented Sep 22, 2023

next time, when you post a PR that you want to close an issue with, you should match a format.
* closes #1234 or * fixes #1234 without any other words

@EvilDragonfiend EvilDragonfiend added this pull request to the merge queue Sep 22, 2023
Merged via the queue into BeeStation:master with commit 98f6a8a Sep 22, 2023
@VickiMorris VickiMorris deleted the surgeryLogging branch September 24, 2023 23:53
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.

[New Feature]: Surgery needs more logging
2 participants