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

Secmodules 2 #436

Merged
merged 49 commits into from
Jan 9, 2025
Merged

Secmodules 2 #436

merged 49 commits into from
Jan 9, 2025

Conversation

fenndragon
Copy link
Contributor

@fenndragon fenndragon commented Dec 27, 2024

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

🆑

  • add: Added 2 new modules
  • add: Quadborg recipes
  • tweak: Tweaked Secborg setup
  • tweak: Tweaked Core map

@github-actions github-actions bot added Status: Needs Review Someone please review this Changes: C# Changes any cs files Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files labels Dec 27, 2024
Copy link
Contributor

github-actions bot commented Dec 27, 2024

RSI Diff Bot; head commit dc12142 merging into 81c0a15
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Floof/Mobs/Silicon/quad.rsi

State Old New Status
quadsprite Modified
quadspriteCC Added

Resources/Textures/Floof/Objects/Specific/Robotics/parts.rsi

State Old New Status
borg_chest+o Added
borg_head+o Added
borg_l_arm+o Added
borg_l_leg+o Added
borg_r_arm+o Added
borg_r_leg+o Added
quad_suit Added

Resources/Textures/Objects/Weapons/Guns/Battery/energy_shotgun.rsi

State Old New Status
base Added
equipped-BACKPACK Added
equipped-SUITSTORAGE Added
icon Added
mag-unshaded-1 Added
mag-unshaded-2 Added
mag-unshaded-3 Added
mag-unshaded-4 Added

Edit: diff updated after dc12142

@FoxxoTrystan FoxxoTrystan added Priority: 3-Low Should be resolved at some point Undergoing Maintainer Discussion This PR is currently going through an internal discussion by the maintainer team. labels Dec 27, 2024
Copy link
Collaborator

@FoxxoTrystan FoxxoTrystan left a 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:

space-wizards/space-station-14#32750

Comment on lines +22 to +26
- type: Body
prototype: Primate
requiredLegs: 1 # TODO: More than 1 leg
- type: Inventory
templateId: Securityborg
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- type: Body
prototype: Primate
requiredLegs: 1 # TODO: More than 1 leg
- type: Inventory
templateId: Securityborg

Copy link
Collaborator

Choose a reason for hiding this comment

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

This is uneeded.

Copy link
Contributor Author

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

Copy link
Collaborator

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.

Copy link
Contributor Author

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.

Comment on lines +49 to +60
id: Securityborg
slots:
- name: pocket1
slotTexture: pocket
slotFlags: POCKET
slotGroup: MainHotbar
stripTime: 3
uiWindowPos: 0,3
strippingWindowPos: 0,4
displayName: Pocket 1
stripHidden: false
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- 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

Copy link
Collaborator

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.

Copy link
Contributor Author

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

Copy link
Contributor

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

Copy link
Contributor Author

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
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do not remove SecWatch

Copy link
Contributor Author

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
Copy link
Collaborator

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.

Copy link
Contributor Author

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

@FoxxoTrystan FoxxoTrystan added Status: Awaiting Changes Do not merge due to requested changes and removed Status: Needs Review Someone please review this labels Dec 27, 2024
/// Spawns a spread of the projectiles when fired
/// </summary>
[RegisterComponent, NetworkedComponent, Access(typeof(SharedGunSystem))]
public sealed partial class ProjectileSpreadComponent : Component
Copy link
Collaborator

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.

Copy link
Contributor Author

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

@fenndragon
Copy link
Contributor Author

all PRS separated and set up

@github-actions github-actions bot added the Changes: Localization Changes any ftl files label Jan 6, 2025
@FoxxoTrystan FoxxoTrystan added Status: Requires This requires something else to be done before being resolved and removed Undergoing Maintainer Discussion This PR is currently going through an internal discussion by the maintainer team. Status: Requires This requires something else to be done before being resolved labels Jan 7, 2025
@FoxxoTrystan FoxxoTrystan self-requested a review January 7, 2025 11:25
@github-actions github-actions bot added Status: Needs Review Someone please review this and removed Status: Awaiting Changes Do not merge due to requested changes labels Jan 7, 2025
Copy link
Collaborator

@Memeji Memeji left a 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.

@Memeji Memeji merged commit 9c91222 into Fansana:master Jan 9, 2025
13 checks passed
Floof-Station-Bot added a commit that referenced this pull request Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Map Changes any yml file in the Maps directories Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files Priority: 3-Low Should be resolved at some point Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants