Security Borg #420
Security Borg #420
Conversation
fenndragon
requested review from
Fansana,
Memeji and
FoxxoTrystan
as code owners
github-actions
bot
added
Status: Needs Review
|
RSI Diff Bot; head commit 364d852 merging into 55d7352 Resources/Textures/Floof/Mobs/Silicon/quad.rsi
Resources/Textures/Floof/Objects/Specific/Robotics/borgmodule.rsi
|
FoxxoTrystan
added
the
Undergoing Maintainer Discussion
Resources/Prototypes/Entities/Objects/Specific/Robotics/borg_modules.yml
Outdated
Show resolved
Hide resolved
Resources/Prototypes/Entities/Objects/Specific/Robotics/borg_modules.yml
Outdated
Show resolved
Hide resolved
Resources/Prototypes/Entities/Objects/Weapons/Guns/Battery/battery_guns.yml
Outdated
Show resolved
Hide resolved
Resources/Textures/Objects/Specific/Robotics/borgmodule.rsi/meta.json
Outdated
Show resolved
Hide resolved
| - type: Construction | ||
| node: cyborg | ||
| - type: Speech | ||
| speechVerb: Robotic |
There was a problem hiding this comment.
| - type: Construction | |
| node: cyborg | |
| - type: Speech | |
| speechVerb: Robotic |
Uneeded and this will cause issues with the current cyborg build, Speechverb is already on parent.
There was a problem hiding this comment.
I literally had to add this back because it threw errors.
There was a problem hiding this comment.
Huh, strange, il look into it.
There was a problem hiding this comment.
A suggestion but you should add IntrinsicRadioTransmitter and ActiveRadio comp so the borg can speak also in security channels.
I suggest those channels: Binary, Common, Security, Science
There was a problem hiding this comment.
I don't know, I think until we get these buildable having them start with security channel might be iffy
There was a problem hiding this comment.
You gave them AccessReader with security access, this borg is a security chassie correct?
There was a problem hiding this comment.
yes, but until they are actually something that may be assembled I think the channel keys should be something put in manually
There was a problem hiding this comment.
This Cyborg is missing/using duplicated sprite for mind state, while this is not an issue right now, when its is public this will need to be changed.
Resources/Prototypes/Floof/Entities/Objects/Specific/Robotics/Borgmodules.yml
Outdated
Show resolved
Hide resolved
Resources/Prototypes/Floof/Entities/Objects/Specific/Robotics/Borgmodules.yml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Ehh this inst normal, did you remove something by mistake?
There was a problem hiding this comment.
Ehh this inst normal, did you remove something by mistake?
There was a problem hiding this comment.
No, I moved an unused sprite into my rsi so it can be used. Because you said to. The image and the module itself both need to be in the rsi
Co-authored-by: FoxxoTrystan <[email protected]>
…Borgmodules.yml Co-authored-by: FoxxoTrystan <[email protected]>
Co-authored-by: FoxxoTrystan <[email protected]>
…Borgmodules.yml Co-authored-by: FoxxoTrystan <[email protected]>
Co-authored-by: FoxxoTrystan <[email protected]>
|
hey i had a security borg made |
FoxxoTrystan
added
Status: Awaiting Changes
Co-authored-by: FoxxoTrystan <[email protected]>
FoxxoTrystan
added
Status: Needs Review
Description
this PR adds Quadborg Security bots to the game as an entity (they have no construction setup or parts built in so it's exclusively an admin only spawn for now, but it also adds a few modules specific to security borgs.
Changelog
🆑