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

Account for configuration channels in NetworkRegistryMixin#preserveSendableChannels #171

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

unilock
Copy link
Contributor

@unilock unilock commented Nov 21, 2024

Fixes client disconnecting from dedicated server with Fabric mods that register configuration phase packets (such as Spell Engine)

…ndableChannels

Fixes client disconnecting from dedicated server with Fabric mods that register configuration phase packets

Signed-off-by: unilock <[email protected]>
@Tapacywka
Copy link

This PR doesn't help solve the issue
image

@unilock
Copy link
Contributor Author

unilock commented Nov 24, 2024

@Tapacywka That's the issue this PR is supposed to fix, and it did appear to fix it for me... I'm not sure what the issue could still be. Hopefully someone who's more knowledgeable about payload registration can shed some insight.

@Su5eD Su5eD added bug Something isn't working question Further information is requested labels Dec 13, 2024
@kolyambo
Copy link

This PR did help to solve ZsoltMolnarrr/SpellEngine#80 for me, just checked on local machine.

@Su5eD Su5eD merged commit 0d25d43 into Sinytra:1.21.1 Jan 29, 2025
1 check passed
@Su5eD
Copy link
Member

Su5eD commented Jan 29, 2025

Thanks for contributing the fix!

TexBlock added a commit to ThinkingStudios/FoxifiedNetworking that referenced this pull request Jan 30, 2025
- Account for configuration channels in NetworkRegistryMixin#preserveSendableChannels ([Sinytra/ForgifiedFabricAPI#171](Sinytra/ForgifiedFabricAPI#171))
TexBlock added a commit to ThinkingStudios/FoxifiedNetworking that referenced this pull request Jan 30, 2025
- Account for configuration channels in NetworkRegistryMixin#preserveSendableChannels ([Sinytra/ForgifiedFabricAPI#171](Sinytra/ForgifiedFabricAPI#171))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants