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

kill some spectator frags #411

Merged
merged 1 commit into from
Jun 21, 2024
Merged

kill some spectator frags #411

merged 1 commit into from
Jun 21, 2024

Conversation

tra
Copy link
Contributor

@tra tra commented Jun 21, 2024

A bunch of spectator frags were being caused by processing the spectator's scout controls and/or not respecting the scout controls of players still in the game.

A bunch of spectator frags were being caused by processing the spectator's scout controls and/or not respecting the scout controls of players still in the game.
@tra tra requested a review from Ymihere03 June 21, 2024 00:05
Copy link
Contributor

@Ymihere03 Ymihere03 left a comment

Choose a reason for hiding this comment

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

Looks good. My original logic didn't cover the right use cases for disabling scout controls.

@Ymihere03 Ymihere03 merged commit c7cc110 into main Jun 21, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants