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

Adds a reliable way to quickly stop a securitron. #16214

Closed
wants to merge 1 commit into from

Conversation

MRLinew
Copy link
Contributor

@MRLinew MRLinew commented Oct 2, 2023

About the PR

If you have access, clicking a securitron while on disarm or grab intent will stop it from moving for a few seconds. This could be seen as buff to antagonists with AA, though securitrons rarely go after people with AA.

Why's this needed?

Right now, the onclick menu for securitrons is useless, as they just move away before you can use it. The only way to stop them is through the slow PDA app. See this forum thread: https://forum.ss13.co/showthread.php?tid=21854&pid=198161#pid198161

Changelog

(u)Froggit Dogget
(+) If you have security access, you can click on a securitron using grab or disarm intent to make it temporarily stop moving/chasing its target.

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 2, 2023
@Sbmhawk
Copy link
Contributor

Sbmhawk commented Oct 2, 2023

Perhaps this could not work for ids set to arrest?

So its more for sec/cap to calm down rabid (non-emagged) beeps

@pgmzeta pgmzeta added C-Feature A new feature or enhancements to existing features A-Game-Objects The point of this PR is to deal with a specific game object labels Oct 4, 2023
@MeggalBozale
Copy link
Contributor

Beepsky only listening to those it deems non-crimers would make sense overall

@TobleroneSwordfish
Copy link
Contributor

Oh I didn't see this, uhh probably not needed now that #16260 is merged? Also intent based interaction with machines + further buffing antags with AA isn't great.

@ZeWaka ZeWaka added the C-Balance Balance changes, buffs and nerfs label Oct 8, 2023
@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Oct 23, 2023
@Tarmunora
Copy link
Member

Oh I didn't see this, uhh probably not needed now that #16260 is merged? Also intent based interaction with machines + further buffing antags with AA isn't great.

@Tarmunora Tarmunora closed this Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Game-Objects The point of this PR is to deal with a specific game object C-Balance Balance changes, buffs and nerfs C-Feature A new feature or enhancements to existing features S-Stale An inactive PR that has had no updates in the past two weeks size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants