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

Fix ghosting in lobby #29648

Closed
wants to merge 6 commits into from
Closed

Conversation

lzk228
Copy link
Contributor

@lzk228 lzk228 commented Jul 1, 2024

About the PR

title
fixes #21695

Why / Balance

bug

Technical details

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase
    Screenshot_20240701_210256

Breaking changes

Changelog

🆑

  • fix: Fixed ghosting in lobby.

Copy link
Contributor

github-actions bot commented Jul 3, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jul 3, 2024
@github-actions github-actions bot removed the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jul 3, 2024
Comment on lines 21 to 23
var player = shell.Player;
var ticker = _entities.System<GameTicker>();

Copy link
Contributor

Choose a reason for hiding this comment

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

We have a system for this now.

Copy link
Contributor

Choose a reason for hiding this comment

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

space-wizards/RobustToolbox#5267

MindSystem also needs updating to use it.

@metalgearsloth metalgearsloth added the S: Awaiting Changes Status: Changes are required before another review can happen label Jul 3, 2024
@lzk228 lzk228 requested a review from metalgearsloth July 4, 2024 14:45
@github-actions github-actions bot added S: Needs Review Status: Requires additional reviews before being fully accepted and removed S: Awaiting Changes Status: Changes are required before another review can happen labels Jul 4, 2024
@lzk228 lzk228 force-pushed the 07-01-fix-ghost-lobby branch from d3fa5ec to 49a0216 Compare July 4, 2024 15:02
@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Aug 26, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@lzk228 lzk228 closed this Oct 24, 2024
@lzk228 lzk228 deleted the 07-01-fix-ghost-lobby branch February 21, 2025 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted S: Needs Review Status: Requires additional reviews before being fully accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

you can hear everyone from station, while being in lobby
2 participants