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

The Security Alert alert button from security PDAs is no longer silent. #6979

Closed
wants to merge 2 commits into from

Conversation

Zonespace27
Copy link
Contributor

@Zonespace27 Zonespace27 commented Dec 24, 2021

[Balance]

About the PR

This PR adds a visible message to when anyone presses the "alert" button with the security alert module in their PDA.

Why's this needed?

Being able to tell when an officer calls for backup (which usually sends the ENTIRE security team scrambling over) is a good thing for the sake of antagonists, since you will no longer be silently ganged up on by 6 security officers when the 1 you cornered hit the button without you knowing.

Changelog

(u)Zonespace
(*)The security alert button (typically found in security PDAs) is no longer completely silent to observers.

@keywordlabeler keywordlabeler bot added the C-Balance Balance changes, buffs and nerfs label Dec 24, 2021
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 24, 2021
@Zonespace27 Zonespace27 added the E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. label Dec 24, 2021
@github-actions
Copy link
Contributor

Created discussion thread: https://forum.ss13.co/showthread.php?tid=17742

@Zonespace27 Zonespace27 marked this pull request as draft December 26, 2021 20:52
@github-actions github-actions bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 1, 2022
@Zonespace27 Zonespace27 changed the title Security PDAs on RP no longer have security alert function. The Security Alert alert button from security PDAs is no longer silent. Jan 1, 2022
@Zonespace27 Zonespace27 marked this pull request as ready for review January 1, 2022 07:00
@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 Jan 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Balance Balance changes, buffs and nerfs E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. 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

1 participant