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

[MIRROR] Post-round blob zombies no longer offer themselves to ghosts #2799

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1882
Original PR: tgstation/tgstation#82542

About The Pull Request

This should properly fix the post-blob-win audio/notif spam for real this time.

I wanted to preserve the effect as much as possible, so they just don't poll for ghosts when the round is already over.

Why It's Good For The Game

Closes #80998 for real this time.

Changelog

🆑 Rhials
fix: The tide of post-round blob zombies when a blob wins will no longer break the speakers in your headset.
/:cl:

…to ghosts (#1882)

* [NO GBP] Post-round blob zombies no longer offer themselves to ghosts (#82542)

## About The Pull Request

This should properly fix the post-blob-win audio/notif spam for real
this time.

I wanted to preserve the effect as much as possible, so they just don't
poll for ghosts when the round is already over.
## Why It's Good For The Game

Closes #80998 for real this time.
## Changelog
:cl: Rhials
fix: The tide of post-round blob zombies when a blob wins will no longer
break the speakers in your headset.
/:cl:

* [NO GBP] Post-round blob zombies no longer offer themselves to ghosts

---------

Co-authored-by: Rhials <[email protected]>
@mogeoko mogeoko merged commit 56966aa into master Apr 10, 2024
26 checks passed
@mogeoko mogeoko deleted the upstream-mirror-1882 branch April 10, 2024 16:56
AnywayFarus added a commit that referenced this pull request Apr 10, 2024
Iajret pushed a commit that referenced this pull request Jun 1, 2024
… as COMSIG_ATOM_DIR_CHANGE (#2799)

* Fixes COMSIG_ATOM_POST_DIR_CHANGE having the same definition as COMSIG_ATOM_DIR_CHANGE (#83604)

## About The Pull Request

Title.
## Why It's Good For The Game

Turns out, signals having the same definition causes a lot of issues.
## Changelog
:cl:
fix: COMSIG_ATOM_POST_DIR_CHANGE is now a functional signal
/:cl:

* Fixes COMSIG_ATOM_POST_DIR_CHANGE having the same definition as COMSIG_ATOM_DIR_CHANGE

---------

Co-authored-by: nikothedude <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
AnywayFarus pushed a commit that referenced this pull request Jun 2, 2024
… as COMSIG_ATOM_DIR_CHANGE (#2799) (#3601)

* Fixes COMSIG_ATOM_POST_DIR_CHANGE having the same definition as COMSIG_ATOM_DIR_CHANGE (#83604)

## About The Pull Request

Title.
## Why It's Good For The Game

Turns out, signals having the same definition causes a lot of issues.
## Changelog
:cl:
fix: COMSIG_ATOM_POST_DIR_CHANGE is now a functional signal
/:cl:

* Fixes COMSIG_ATOM_POST_DIR_CHANGE having the same definition as COMSIG_ATOM_DIR_CHANGE

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: nikothedude <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
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.

3 participants