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 consequences to some (funny) actions #2719

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Bokkiewokkie
Copy link
Contributor

About The Pull Request

Adds admin logging to the BSA being fired at unreasonable times.

Why It's Good For The Game

Having a quick popup and game logs for admins to see WHO just blew up half of the ship is helpful.

Testing Photographs and Procedure

image

Changelog

🆑
add: Added a peculiar effect when an unreasonable amount of energy hits a singularity generator.
admin: Added admin logging for BSA misfire incidents
/:cl:

new /obj/anomaly/singularity(location, 2500)
var/turf/overmaplocation = get_turf(get_overmap())
if(overmaplocation)
new /obj/effect/overmap_anomaly/singularity(overmaplocation)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should link this anomaly to the ship's current system (and probably just drop it into the aether if in hyperspace?) or it'll not do anything (due to the change making them use system contents)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I figured it wouldn't be necessary because this will instantly end the round

Copy link
Contributor

@DeltaFire15 DeltaFire15 Nov 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weeeelll it wouldn't since overmap singularities go by system contents to see what they affect in what way.. which, if there isn't any present, the ship is going to be perfectly fine (but I guess you could just call the overmap singularities procs instead if you do not feel like linking it)

Copy link
Contributor Author

@Bokkiewokkie Bokkiewokkie Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean when I tested it the game immediately deleted the ship sooo....
(I will add proper stuff to it as well anyway)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants