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

Reorganize roles and add warning about the two job rule #350

Merged
merged 3 commits into from
Aug 10, 2024

Conversation

dffdff2423
Copy link
Collaborator

@dffdff2423 dffdff2423 commented Aug 4, 2024

About the PR

  • Recategorize jobs to fit their use on CD
  • Warning about job rule for HumanoidProfileEditor

Why / Balance

This is just a ""temporary"" measure until somebody enforces this mechanically. See #347

Media

image

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

🆑

  • add: Reorganize the character editor to make it clear what jobs are in what department for the "two department + service" rule

- Psych (should not be) station specific
- They do not reflect how our two departments + service work.
@Nairodian
Copy link
Collaborator

How does this look on the crew manifest, you know?

@dffdff2423
Copy link
Collaborator Author

dffdff2423 commented Aug 5, 2024

image
Let me know if you want one with more characters.

I think not having another command department is fine because it is obvious all of the department heads are a head from the blue icon and the name. It would be trivial to make high command it's own department in addition to command if you would prefer that. I am a bit mixed on if the HoP should be listed in service and High Command.

The other thing I just realized now is that this structure could imply that the HoP is above the other heads when they are really just "first among equals"

@Nairodian
Copy link
Collaborator

Hm, I realize it's not too bad either way. Being able to quickly take a glance at the manifest and get a list of all the command members was nice, but perhaps just ask on the Discord what others think.

@LankLTE
Copy link
Contributor

LankLTE commented Aug 7, 2024

I'm with Nairod that it's nice to just be able to look and see every command member

@dffdff2423
Copy link
Collaborator Author

dffdff2423 commented Aug 7, 2024

Then I think re-adding Command but then also adding "High Command" might be the best approach. If we do go with this approach, should the HoP be removed from service on the list?

@dffdff2423
Copy link
Collaborator Author

Image of current state:
image

@LankLTE LankLTE merged commit 7f27348 into cosmatic-drift-14:master Aug 10, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants