-
Notifications
You must be signed in to change notification settings - Fork 345
PDA Alert level #6454
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
PDA Alert level #6454
Conversation
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.
link and credit the original PR you ported, please.
Ah sorry, I had found the code raw so hadn't considered looking for it. |
#define ALERT_RELEVANCY_SAFE 0 /// * 0: User is not in immediate danger and not needed for some station-critical task. | ||
#define ALERT_RELEVANCY_WARN 1 /// * 1: Danger is around, but the user is not directly needed to handle it. | ||
#define ALERT_RELEVANCY_PERTINENT 2 /// * 2: Danger is around and the user is responsible for handling it. | ||
/obj/item/modular_computer/proc/get_security_level_relevancy() |
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.
ngl I'd prefer these checks be a part of /datum/security_level
instead
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.
honestly i'll refactor in a later PR, this has been TM'd long enough
About The Pull Request
Ports alert levels will showing on PDAS from #89173

The intensity will also change depending on how relevant it is to the users access.
Why It's Good For The Game
Visual communication of ongoing levels is great. Also lets me know if I can slip a security officer for having a gun out on green alert.
Changelog
🆑
qol: Alert levels now show on pda's
/:cl: