Chain of Command Tweaks, (Fixes #12556 and makes #12723 redundant + More.) - #12724
Open
neeshacark wants to merge 1 commit into
Open
Chain of Command Tweaks, (Fixes #12556 and makes #12723 redundant + More.)#12724neeshacark wants to merge 1 commit into
neeshacark wants to merge 1 commit into
Conversation
cmss13-ci
Bot
added
Feature
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
About the pull request
Puts SO as fourth in command, coming after ASO and before CMO and CMP. Also makes Field Doctor after normal Doctor in the chain (meaning this PR fixes #12556 and makes #12723 redundant.
This is the conclusion of my ideaguys post (which I made to gauge community opinion) https://forum.cm-ss13.com/t/move-so-up-two-steps-in-chain-of-command-meaning-they-roll-aco-before-cmo-and-cmp/21085/14 and adding in a fix from Jackie's PR since it touches the same code.
Explain why it's good for the game
As said in my ideaguys post, there is zero reason we should be making people who joined the game to command CIC (SO's) be rolling aCO AFTER people who joined to manage medical and teach doctors. The skills don't overlap, CMO players don't need (or want to) play CIC to unlock the role. It's not their field. SO players on the other hand literally join to do this. Same goes for CMP. Yes, they're senior command but entirely different fields. Also, as was mentioned in the post, most CMOs will immediately pass off the command to someone else - like an SO. So at that rate this is just QoL
As far as the FD, no reason they should be rolling above doctors. They're a role meant to deploy, they join the round expecting do. Doctors don't. No reason for FD to be above normal doctors in CoC.
For further reasons, review community opinion on my aforementioned ideaguys post: https://forum.cm-ss13.com/t/move-so-up-two-steps-in-chain-of-command-meaning-they-roll-aco-before-cmo-and-cmp/21085/14
Testing Photographs and Procedure
Screenshots & Videos
Put screenshots and videos here with an empty line between the screenshots and the
<details>tags.Changelog
🆑
add: Staff Officers now come fourth in the chain of command, and Field Doctors are after normal Doctors.
/:cl: