-
Notifications
You must be signed in to change notification settings - Fork 92
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
Secmodules 2 #436
Secmodules 2 #436
Conversation
RSI Diff Bot; head commit dc12142 merging into 81c0a15 Resources/Textures/Floof/Mobs/Silicon/quad.rsi
Resources/Textures/Floof/Objects/Specific/Robotics/parts.rsi
Resources/Textures/Objects/Weapons/Guns/Battery/energy_shotgun.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.
If you wish borg the ability to strip i suggest adding this PR:
- type: Body | ||
prototype: Primate | ||
requiredLegs: 1 # TODO: More than 1 leg | ||
- type: Inventory | ||
templateId: Securityborg |
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.
- type: Body | |
prototype: Primate | |
requiredLegs: 1 # TODO: More than 1 leg | |
- type: Inventory | |
templateId: Securityborg |
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 uneeded.
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.
its required to give them the pocket to put handcuffs in
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.
Then make a custom item/tool for borgs to cuff someone.
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 has already been discussed with floof, cuffs in a module don't work, I already looked into an alternative for this.
id: Securityborg | ||
slots: | ||
- name: pocket1 | ||
slotTexture: pocket | ||
slotFlags: POCKET | ||
slotGroup: MainHotbar | ||
stripTime: 3 | ||
uiWindowPos: 0,3 | ||
strippingWindowPos: 0,4 | ||
displayName: Pocket 1 | ||
stripHidden: false |
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.
- type: inventoryTemplate | |
id: Securityborg | |
slots: | |
- name: pocket1 | |
slotTexture: pocket | |
slotFlags: POCKET | |
slotGroup: MainHotbar | |
stripTime: 3 | |
uiWindowPos: 0,3 | |
strippingWindowPos: 0,4 | |
displayName: Pocket 1 | |
stripHidden: false |
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.
We not allowing to provide borgs with pockets/inventory.
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 pocket is required to allow the borg to be able to apprehend with cuffs
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.
umm when I made a borg you didn't need any of that just need to put the cuff in the mod and it was useable
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 cuffs act like they can be used but they will not actually cuff people if you just stick them in the module
- SecWatchCartridge | ||
- LogProbeCartridge |
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.
Do not remove SecWatch
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.
there is presently a bug that doesn't allow logprobe to be installed
items: | ||
- ForensicScanner | ||
- DetectivePDA |
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.
Giving a full PDA seem just... bad, there should be another way.
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.
to my knowledge there is not to be able to allow them to utilize logprobing with doors
/// Spawns a spread of the projectiles when fired | ||
/// </summary> | ||
[RegisterComponent, NetworkedComponent, Access(typeof(SharedGunSystem))] | ||
public sealed partial class ProjectileSpreadComponent : Component |
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 component is unused. I don't see why it's here.
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.
it honestly shouldn't be, I could've sworn I removed it
all PRS separated and set up |
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.
Told this has been accepted by Floof and passed tests.
Description
Adds a couple more Security Borg modules to the game, one which is for detective work, and one for heavy assault/combat.
I've also ported the Energy Shotgun from Wizden with this PR
Changelog
🆑