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

Auxiliary Support Officer #4081

Merged
merged 18 commits into from
Aug 7, 2023
Merged

Conversation

morrowwolf
Copy link
Member

@morrowwolf morrowwolf commented Aug 1, 2023

About the pull request

This PR adds the Auxiliary Support Officer.

Read more here: https://docs.google.com/document/d/1aZLVaIwopVj_tant1Wcjuqo0_Ov70IL3U-UHfg7P7wo/edit?usp=sharing

Main points:
Auxiliary Support Officer oversees the hangar, the intel team, the engineering department, and the requisitions department.

Chief Engineer has been demoted to O1.

Requisitions Officer renamed to Quartermaster and is now a Gunnery Sergeant.

For the duration of testing the following is true:

The Auxiliary Support Officer may not deploy.

The Chief Engineer will be given explicit authority over construction and control of the Forward Operating Base. The Chief Engineer, and Maintenance Technicians under them, may deploy freely if there is an Auxiliary Support Officer. If there is not an Auxiliary Support Officer the Chief Engineer may deploy with permission from the Acting Commander and Maintenance Technicians may deploy with the permission of the Chief Engineer. The Chief Engineer is not required to assist with FOB construction unless ordered to. One member of Engineering, or the Auxiliary Support Officer, should be aboard the ship at all times.

The Quartermaster will have unchanged deployment standards as the Requisitions Officer. Cargo Technicians may deploy if given permission by the Quartermaster, Auxiliary Support Officer, or aCO. The Mess Technician will be formally put under the Quartermaster in Requisitions.

Explain why it's good for the game

Explained in above linked documentation.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 Morrow, Thwomper
add: Added Auxiliary Support Officer
add: Auxiliary Support Officer oversees the hangar, the intel team, the engineering department, and the requisitions department
add: Chief Engineer has been demoted to O1.
add: Requisitions Officer is now "Quartermaster" and a Gunnery Sergeant.
/:cl:

@github-actions github-actions bot added Sprites Remove the soul from the game. Mapping did you remember to save in tgm format? Feature Feature coder badge labels Aug 1, 2023
@Ben10083
Copy link
Contributor

Ben10083 commented Aug 2, 2023

add: Requisitions Officer is now "Quartermaster" and a Gunnery Sergeant. /🆑

I know this is a nitpick, but recommend renaming this to "Requisitions Officer renamed to Quartermaster and demoted to Gunnery Sergeant.

Though wasnt RO same rank as CE? Why not just demote it to O1 as well

Copy link
Contributor

@FransFeiffer FransFeiffer left a comment

Choose a reason for hiding this comment

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

Sprite's look good, appro:b:ed

@FransFeiffer FransFeiffer added the Sprites Approved confirmed no stray pixels label Aug 2, 2023
@Brewstone

This comment was marked as off-topic.

@morrowwolf morrowwolf marked this pull request as ready for review August 6, 2023 09:28
Copy link
Member

@harryob harryob left a comment

Choose a reason for hiding this comment

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

code

@harryob harryob added this pull request to the merge queue Aug 7, 2023
Merged via the queue into cmss13-devs:master with commit 583911a Aug 7, 2023
26 checks passed
cm13-github added a commit that referenced this pull request Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge Mapping did you remember to save in tgm format? Sprites Approved confirmed no stray pixels Sprites Remove the soul from the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants