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 Talking Across Docked Grids #593

Merged
merged 2 commits into from
Jul 26, 2024

Conversation

VMSolidus
Copy link
Member

Description

#574 introduced a bug whereby people who are each on separate grids, but the grids were attached, could not talk to each other. I have corrected this by making it so that when the Cross-Grid check is handled, it also checks to see if the two different grids are JOINTed to each other. Therefore allowing sound to travel across the connection. This should also work from Shuttle To Planet, since it's also handled via the same system. This means that docked shuttles allow sound to travel across to the docked station(or other shuttle), as well as shuttles docked to a planet's surface will permit sound to travel to the planet.

Changelog

🆑

  • fix: Fixed a bug where sound was not traveling over a Shuttle-Docking connection. Attached grids now permit sound to travel through.

@github-actions github-actions bot added the Changes: C# Changes any cs files label Jul 25, 2024
Content.Server/Chat/Systems/ChatSystem.cs Outdated Show resolved Hide resolved
Content.Server/Chat/Systems/ChatSystem.cs Outdated Show resolved Hide resolved
@DEATHB4DEFEAT DEATHB4DEFEAT added Priority: 2-High Needs to be resolved as soon as possible Size: 5-Very Small For especially small issues/PRs Type: Bug [Un]intentional issue with something labels Jul 25, 2024
Co-authored-by: DEATHB4DEFEAT <[email protected]>
Signed-off-by: VMSolidus <[email protected]>
@OldDanceJacket OldDanceJacket merged commit ec76fb9 into Simple-Station:master Jul 26, 2024
11 checks passed
SimpleStation14 added a commit that referenced this pull request Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Priority: 2-High Needs to be resolved as soon as possible Size: 5-Very Small For especially small issues/PRs Type: Bug [Un]intentional issue with something
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants