[MIRROR] re-adds list of components for admins to remove #2757
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* re-adds list of components for admins to remove (#82461) ## About The Pull Request The list of components on a mob when admins try to remove one didn't actually show them, now it does.  ## Why It's Good For The Game Messing with components/elements on mobs are such a pain, in this case was broken entirely.  ## Changelog :cl: admin: Removing components button now lists components to remove /:cl: * re-adds list of components for admins to remove --------- Co-authored-by: John Willard <[email protected]>
AnywayFarus
added a commit
that referenced
this pull request
Apr 6, 2024
Iajret
pushed a commit
that referenced
this pull request
Jun 21, 2024
* Telecomms Specialist Adds a new lightweight role to engineering, the Telecomms Specialist! Monitor and repair telecomms when needed, watch out for suspicious activity on the network, and report to both engineering and security. * Removed security comms And changed alt job titesl * minor issues fixed * Update Added some basic-edited uniforms, changed alternate job titles again. * oopsie forgot a comma * jacket gives them a jacket, it's cold in telecomms' machine room * alt titles and icons * upped to 2 slots * added a tcomm specific jacket * added span_doyourjobidiot A warning/friendly reminder to abide by server rules. * back down to 1 * added crew monitor bit and adjusted jumpsuit/skirt variables and desc * mapping spawnpoint * landmarks and spawn area * fixed * Changes as requested * added another area maybe this will get it to pass map integration * another try does a fourth area even work * removed unneeded spawn area since we're gonna have to manually place it anyway... * added landmarks for nova maps via lt3 * Apply suggestions from code review * Apply suggestions from code review --------- Co-authored-by: Zergspower <[email protected]> Co-authored-by: Bloop <[email protected]>
Iajret
added a commit
that referenced
this pull request
Jun 22, 2024
* [SEMI-MODULAR] Telecomms Specialist (#2757) * Telecomms Specialist Adds a new lightweight role to engineering, the Telecomms Specialist! Monitor and repair telecomms when needed, watch out for suspicious activity on the network, and report to both engineering and security. * Removed security comms And changed alt job titesl * minor issues fixed * Update Added some basic-edited uniforms, changed alternate job titles again. * oopsie forgot a comma * jacket gives them a jacket, it's cold in telecomms' machine room * alt titles and icons * upped to 2 slots * added a tcomm specific jacket * added span_doyourjobidiot A warning/friendly reminder to abide by server rules. * back down to 1 * added crew monitor bit and adjusted jumpsuit/skirt variables and desc * mapping spawnpoint * landmarks and spawn area * fixed * Changes as requested * added another area maybe this will get it to pass map integration * another try does a fourth area even work * removed unneeded spawn area since we're gonna have to manually place it anyway... * added landmarks for nova maps via lt3 * Apply suggestions from code review * Apply suggestions from code review --------- Co-authored-by: Zergspower <[email protected]> Co-authored-by: Bloop <[email protected]> * tcomm admin added --------- Co-authored-by: SciDragon <[email protected]> Co-authored-by: Zergspower <[email protected]> Co-authored-by: Bloop <[email protected]> Co-authored-by: Iajret <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Mirrored on Nova: NovaSector/NovaSector#1829
Original PR: tgstation/tgstation#82461
About The Pull Request
The list of components on a mob when admins try to remove one didn't actually show them, now it does.
Why It's Good For The Game
Messing with components/elements on mobs are such a pain, in this case was broken entirely.
Changelog
🆑 JohnFulpWillard
admin: Removing components button now lists components to remove
/:cl: