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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rewrites alarm handler for multi z functionality #20404
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.
I'll note that alarms from legit outposts will no longer be detected.
I mean, it does still make sense. How would a simple, low-power alarm on a faraway outpost transmit all the way back to the Torch? |
The same way radio communication does? |
Alerts from outposts (if we had any) currently already do not show on the modular console. I could add their z levels to the loop. They'd be visible from anywhere, but that would ICly just imply they have a strong signal. Does that sound good? |
@@ -4,7 +4,6 @@ | |||
nanomodule_path = /datum/nano_module/alarm_monitor/engineering | |||
ui_header = "alarm_green.gif" | |||
program_icon_state = "alert-green" | |||
program_key_state = "atmos_key" |
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.
???
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.
Not sure why that happened. Fixed.
馃啈
bugfix: Fixes alert monitors being red for away mission alerts.
bugfix: Fixes nonmodular atmos/alert consoles showing away mission alerts.
/:cl: