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 #2256

Merged
merged 2 commits into from
Apr 30, 2024
Merged

Conversation

NovaBot13
Copy link
Collaborator

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

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

jlsnow301 and others added 2 commits April 30, 2024 12:14
## 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:
@NovaBot13 NovaBot13 added Fix Rewrites a bug so it appears in different circumstances GBP: No Update Prevent changes to the user's GBP balance from this PR UI We make the game less playable, but with round edges labels Apr 30, 2024
@vinylspiders vinylspiders enabled auto-merge (squash) April 30, 2024 16:26
@vinylspiders vinylspiders merged commit 8784627 into master Apr 30, 2024
23 of 24 checks passed
@vinylspiders vinylspiders deleted the upstream-merge-82931 branch April 30, 2024 17:46
NovaBot13 added a commit that referenced this pull request Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Rewrites a bug so it appears in different circumstances GBP: No Update Prevent changes to the user's GBP balance from this PR UI We make the game less playable, but with round edges
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants