Skip to content

Readyup - Fixup and refactor checks (Stack PR 2 of 2) - #2047

Open
nullsystem wants to merge 2 commits into
NeoUIScoreboardReadyUpCheckfrom
NeoGamemodeFixupReadyUpCheck
Open

Readyup - Fixup and refactor checks (Stack PR 2 of 2)#2047
nullsystem wants to merge 2 commits into
NeoUIScoreboardReadyUpCheckfrom
NeoGamemodeFixupReadyUpCheck

Conversation

@nullsystem

Copy link
Copy Markdown
Collaborator

Description

Make sure it only triggers readyup for gamemodes that deals with
readyup, if not, just go on as if readyup not a thing.

Toolchain

  • Linux GCC Distro Native Arch/GCC 16

Linked Issues

  • fixes #
  • related #

Stack created with GitHub Stacks CLIGive Feedback 💬

Make sure it only triggers readyup for gamemodes that deals with
readyup, if not, just go on as if readyup not a thing.
@nullsystem nullsystem changed the title Readyup - Fixup and refactor checks Readyup - Fixup and refactor checks (Stack 2 of 2) Aug 3, 2026
@nullsystem nullsystem changed the title Readyup - Fixup and refactor checks (Stack 2 of 2) Readyup - Fixup and refactor checks (Stack PR 2 of 2) Aug 3, 2026
@nullsystem
nullsystem requested a review from a team August 3, 2026 21:19
AdamTadeusz
AdamTadeusz previously approved these changes Aug 11, 2026
@AdamTadeusz
AdamTadeusz requested a review from a team August 11, 2026 10:23
@nullsystem

Copy link
Copy Markdown
Collaborator Author

@AdamTadeusz Fixed readyup shown for non-playing players: 43086fc

@nullsystem nullsystem added the New Changes for Reviewers New changes has been made. This label is mostly to notify reviewers. label Aug 13, 2026
@AdamTadeusz

Copy link
Copy Markdown
Contributor

usually we check if a player is on a playing team by comparing against FIRST_PLAYING_TEAM

@AdamTadeusz
AdamTadeusz requested a review from a team August 14, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New Changes for Reviewers New changes has been made. This label is mostly to notify reviewers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants