-
Notifications
You must be signed in to change notification settings - Fork 400
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
TerraGov Inspector #2791
base: master
Are you sure you want to change the base?
TerraGov Inspector #2791
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
RSI Diff Bot; head commit 89ca87c merging into 069dc1e Resources/Textures/DeltaV/Clothing/Ears/Headsets/terragov.rsi
Resources/Textures/DeltaV/Clothing/Head/Hats/beret_terragov.rsi
Resources/Textures/DeltaV/Clothing/Head/Hats/terracap.rsi
Resources/Textures/DeltaV/Clothing/Uniforms/Jumpsuit/terragov.rsi
|
Honestly they should be focusing on making sure the station is upholding the law |
Are laws concerning worker safety and workplace sanitation non existent? :blunt: |
I really like this concept! Also, TerraGov handyflag, one day...
|
Personally, I'd prefer to avoid adding these things to make it easier for syndicate agents to impersonate an inspector. A legitimate inspector can still write their own inspection warrant on the shuttle before they arrive to the station and stamp it with their stamp. Since they have command access in their ID, they can also walk into the bridge and use the announcement console to make their presence known. |
Fair point. I dont think even the most ornery captain would think "Yes, I can tell the man who centcom has given command access to leave." CentCom ops get insta-"valided", so this would be a nice middle ground. |
there are no space workspace safety laws so this should be a super lawyer making sure all space law violations are correctly punished imo |
Direction is looking it over. This one has big implications so it'll probably be several days, but we'll pop back in whenever we have relevant feedback. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! I personally really love this concept, and think the contents bring a lot of options to our Curator team with TerraGov flavored events.
I commented a few issues, the common theme is effectively that we're not sure how this fits as an antagonist, and we're also worried it becomes routine and expected- similar to how the traveling chef is ignored in 90% of cases. We're curious if you have any ideas in how they might be driven to interact further with the station?
Some ideas we've had is a loose re framing of this to Internal Affairs, with objectives to secretly kidnap and interview crewmembers, or acting more as a vigilante to hunt down a specific Fugitive.
I'd be curious to hear your thoughts, and look forward to seeing this evolve!
Resources/Locale/en-US/deltav/game-ticking/game-rules/rule-inspector.ftl
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensure four space indentation on your JSON files, leave the upstream ones with their current indentation.
- type: CartridgeLoader | ||
preinstalled: | ||
- CrewManifestCartridge | ||
- NotekeeperCartridge | ||
- NewsReaderCartridge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is redundant, you inherit these from the BasePDA
.
- type: Item | ||
heldPrefix: default |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is redundant due to the IDCardStandard
parenting.
- type: CommandStaff | ||
- !type:AddComponentSpecial | ||
components: | ||
- type: PsionicBonusChance #Nyano - Summary: makes it more likely to become psionic. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the Nyano comment.
parent: JobIcon | ||
id: JobIconTerraGov | ||
icon: | ||
sprite: /Textures/DeltaV/Interface/Misc/job_icons.rsi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The /Textures/
is extra.
## DeltaV - TerraGov Clipboard | ||
- type: entity | ||
id: BoxFolderTerraGovClipboard | ||
parent: BoxFolderClipboard | ||
name: TerraGov clipboard | ||
description: A luxurious clipboard upholstered with blue velvet. Often seen carried by TerraGov officials, seldom seen actually used. | ||
components: | ||
- type: Sprite | ||
sprite: DeltaV/Objects/Misc/terragov_clipboard.rsi | ||
layers: | ||
- state: clipboard | ||
- state: clipboard_paper | ||
map: ["clipboard_paper"] | ||
visible: false | ||
- state: clipboard_pen | ||
map: ["clipboard_pen"] | ||
visible: false | ||
- state: clipboard_over | ||
- type: Item | ||
sprite: DeltaV/Objects/Misc/terragov_clipboard.rsi | ||
- type: Clothing | ||
sprite: DeltaV/Objects/Misc/terragov_clipboard.rsi | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move this to the _DV
namespace.
- type: entity | ||
name: terragov pen | ||
parent: Pen | ||
id: TerraGovPen | ||
description: A fancy and bright blue pen that marks you as official TerraGov paper-pusher. | ||
components: | ||
- type: Sprite | ||
state: terragov_pen | ||
- type: Item | ||
heldPrefix: terragov_pen | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move this to the _DV
namespace.
Just in case, here is an example of a correct JSON indentation: Delta-v/Resources/Textures/_DV/Clothing/Uniforms/Jumpsuit/security_alt.rsi/meta.json Lines 1 to 30 in 47749d9
|
Direction update, we're still workshopping this with the author. Discussion is in a thread in the contrib channel on Discord |
Since a direction doesn't want a TerraGov themed inspector any longer, I'll be removing all role related changes and make this PR purely for the sprites so they can be used for events (and maybe available in an emag'd uniform printer). I am dismissing all the feedback regarding the files that touch upon the role, since it is not relevant any longer. |
About the PR
Adds the TerraGov Inspector as a ghost role antagonist.
Why / Balance
This is meant to be a fun RP heavy antagonist that's different from existing antagonists with no mechanical depth. You tie up station's attention and resources with your presence, inspect departments and interview the criminally underpaid NanoTrasen employees about their working conditions.
The inspector's outfit/id is also chameleon whitelisted to enable syndicate impostors that are less valid than the usual CentCom impostor.
The inspector currently uses the same shuttle as the syndicate recruiter and spawns with:
Technical details
To Do:
There's still a few kinks I need to iron out:
Media
Sprite showcase
Ghostrole/Shuttle
Objectives
Stamp
Requirements
Breaking changes
My sanity.
Changelog
🆑