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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIRROR] [no gbp] Fixes deathmatch observers #3127

Merged
merged 1 commit into from
Apr 30, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#2256
Original PR: tgstation/tgstation#82931

About The Pull Request

Got a little ahead of myself. My apologies. Fixes some cases in the deathmatch UI that cause blue screens / improper perms.

I've also made deathmatch send an actual array rather than an object. Don't make javascript iterate object.keys please. Fixed the types surrounding this.

Why It's Good For The Game

Fixes #82926

Changelog

馃啈 jlsnow301
fix: Fixes a bluescreen in the deathmatch lobby UI.
/:cl:

* [no gbp] Fixes deathmatch observers (#82931)

## About The Pull Request
Got a little ahead of myself. My apologies. Fixes some cases in the
deathmatch UI that cause blue screens / improper perms.

I've also made deathmatch send an actual array rather than an object.
Don't make javascript iterate object.keys please. Fixed the types
surrounding this.
## Why It's Good For The Game
Fixes #82926
## Changelog
:cl:
fix: Fixes a bluescreen in the deathmatch lobby UI.
/:cl:

* [no gbp] Fixes deathmatch observers

---------

Co-authored-by: Jeremiah <42397676+jlsnow301@users.noreply.github.com>
@mogeoko mogeoko merged commit 19bc36a into master Apr 30, 2024
27 checks passed
@mogeoko mogeoko deleted the upstream-mirror-2256 branch April 30, 2024 22:24
AnywayFarus added a commit that referenced this pull request May 1, 2024
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.

None yet

3 participants