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

Erm.... No More Foreigner Trait Head Roles !!!!!!!!! #1670

Merged
merged 10 commits into from
Jan 27, 2025

Conversation

SixplyDev
Copy link
Contributor

@SixplyDev SixplyDev commented Jan 27, 2025

Description

This PR prevents heads of staff and security members from having foreigner traits, muted trait, blindness trait, and pacifist trait (only HoS & Cap cannot be pacifist). These traits are major disabilities and actively detrimental to their departments & the rest of command staff if they have them. These traits make them unable to clearly and effectively communicate or render them unable to do a critical part of their jobs in the case of pacifist and muted.


TODO

  • Security Department can no longer have pacifist, muted, blind, brittle bone disease, or foreigner traits.
  • Head of Security can no longer have pacifist, muted, blind, brittle bone disease, or foreigner traits.
  • Heads of staff can no longer have muted, blind, or foreigner traits.
  • The Captain can no longer have pacifist, muted, blind, brittle bone disease, or foreigner traits.
  • Head of Personnel can no longer have muted, blind, or foreigner traits.

Changelog

🆑 ShirouAjisai

  • tweak: Security Department can no longer have pacifist, muted, blind, brittle bone disease, or foreigner traits.
  • tweak: Head of Security can no longer have pacifist, muted, blind, brittle bone disease, or foreigner traits.
  • tweak: Heads of staff can no longer have foreigner traits.
  • tweak: The Captain can no longer have pacifist or foreigner traits.
  • tweak: The Head of Personnel can no longer have foreigner traits.
  • tweak: The Administrative Assistant can no longer have foreigner traits.

@github-actions github-actions bot added Status: Needs Review Someone please review this Changes: YML Changes any yml files labels Jan 27, 2025
@sleepyyapril
Copy link
Contributor

sleepyyapril commented Jan 27, 2025

Good RP and admins fix this problem entirely (with regards to blindness and foreigner, anyway). AHelp when you encounter someone who can't do the former.

@VMSolidus
Copy link
Member

This PR definitely needs to be significantly trimmed down. I'm also in agreement with my Maintainers that the list really should be significantly pared down.

Realistically, CMO, Mystagogue, CE, and Logistics Officer it genuinely doesn't matter if they have disabilities or are pacifists. I'm not strongly in agreement with them over discluding the Foreigner traits, since unlike Mute it outright prevents understanding your underlings. Not being fluent in the corporation's language is not the same as a disability, its a lack of a vital skill.

As one of the "Mute character mains", I can attest that you absolutely can be an excellent Captain while being mute, and it involves judicious use of the comms console and fax machine.

Of the command roles, the only one where an extremely strict no-major disabilities requirement makes sense is the Head Of Security. The HoS(and Security in general) needs to be able to pass a very stringent physical and mental aptitude test, because NT needs to be able to trust them with firearms, and be physically capable of defending corporate assets. So my request is, "both Foreigner traits" are fine to exclude from all command, but only HoS(Plus most security roles) actually needs strict ability requirements.

@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Jan 27, 2025
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Jan 27, 2025
@VMSolidus
Copy link
Member

You can lump in Blueshield with Security, since as a bodyguard their job description includes "Ass kicking" as a required skill. But Centcomm Rep wouldn't be lumped in because they're really just a paper pusher that might be armed.

@SixplyDev
Copy link
Contributor Author

Ok should be ready for review now

@SixplyDev SixplyDev requested a review from VMSolidus January 27, 2025 17:33
ShirouAjisai and others added 2 commits January 27, 2025 11:36
Oops thanks, got readded when I was trying to fix the merge conflict

Co-authored-by: sleepyyapril <[email protected]>
Signed-off-by: SixplyDev <[email protected]>
@VMSolidus VMSolidus merged commit a2c2fc0 into Simple-Station:master Jan 27, 2025
10 of 11 checks passed
@SixplyDev SixplyDev deleted the erm.-command-fuckery branch January 27, 2025 17:45
SimpleStation14 added a commit that referenced this pull request Jan 27, 2025
@hivehum
Copy link

hivehum commented Jan 28, 2025

howdy, i'm a serial lurker / maint for a fork that occasionally ports from EE and i just wanted to pop in and say i'm really surprised this got merged? my understanding of the design ethos of einstein engines is to prioritize more interesting roleplay opportunities over minmaxing and this seems like... fully contradictory to that. these traits are outright disadvantageous to playing these roles so someone who opts into them is opting into a complex and interesting RP experience. like i dont frequently play on forks that i don't work on so maybe this is some kind of chronic issue i've completely missed but this really seems like something that would be better handled by downstreams...?

@SixplyDev
Copy link
Contributor Author

SixplyDev commented Jan 28, 2025

howdy, i'm a serial lurker / maint for a fork that occasionally ports from EE and i just wanted to pop in and say i'm really surprised this got merged? my understanding of the design ethos of einstein engines is to prioritize more interesting roleplay opportunities over minmaxing and this seems like... fully contradictory to that. these traits are outright disadvantageous to playing these roles so someone who opts into them is opting into a complex and interesting RP experience. like i dont frequently play on forks that i don't work on so maybe this is some kind of chronic issue i've completely missed but this really seems like something that would be better handled by downstreams...?

The biggest issue I saw and the reason I did this is because people would take these traits and then drag down the rest of the team with them. I personally have no issues with someone being a mute or foreigner member of a department because I think it leads to excellent RP opportunities but I personally feel like a line has to be drawn between interesting RP and not allowing people in very important roles to drag everyone down with them because they can't handle their disabilities. It's an easy fix if you'd like to change this on your downstream fork.

@hivehum
Copy link

hivehum commented Jan 28, 2025

The biggest issue I saw and the reason I did this is because people would take these traits and then drag down the rest of the team with them. I personally have no issues with someone being a mute or foreigner member of a department because I think it leads to excellent RP opportunities but I personally feel like a line has to be drawn between interesting RP and not allowing people in very important roles to drag everyone down with them because they can't handle their disabilities. It's an easy fix I'd you'd like to change this on your downstream fork.

idk ive had such really cool and memorable experiences interacting with pacifist and blind members of security that this fully seems like something that would be better handled through ahelp rather than slapping the entire mechanic out of the hands of the playbase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants