kikugie / elytra-trims

Lightweight and customizable support for trimmable elytras
https://modrinth.com/mod/elytra-trims
GNU Lesser General Public License v3.0
23 stars 8 forks source link

[Bug]: Servers require client side mod installation on 1.20.5+ #56

Closed sternschnaube closed 1 month ago

sternschnaube commented 1 month ago

Mod loader

Fabric

Minecraft version

1.20.6

Mod version

3.1.4

Affected mods

Bug description

image

Received unknown remote registry entries from server Registry entry (minecraft:crafting_special_elytrapatterns) is missing from local registry (minecraft:recipe_serializer) Registry entry (minecraft:crafting_special_elytraglow) is missing from local registry (minecraft:recipe_serializer) Registry entry (minecraft:crafting_special_elytraanimation) is missing from local registry (minecraft:recipe_serializer)

Steps to reproduce

No response

Relevant logs

No response

kikugie commented 1 month ago

Seems like your server has requireClientSide option enabled, in which case it is an expected error.

sternschnaube commented 1 month ago

Using the new generated config, in which this option is from the beginning disabled.

{"addGlow":true,"cleanableElytra":true,"requireClientSide":false,"addTrims":true,"addPatterns":true}

kikugie commented 1 month ago

Please verify that it's not affected by any other mods you have. I can't reproduce it with a local server running elytra trims 3.1.4 and a fully vanilla client: image

sternschnaube commented 1 month ago

The issue is the Fabric API itself. Very weird, vanilla connects without problems. If Fabric API is present on the client, the problem occurs.

kikugie commented 1 month ago

Weird indeed, I'll look into it later today

sternschnaube commented 1 month ago

Maybe it's related with this.

image

kikugie commented 1 month ago

This is gonna be more work than I anticipated, blame 1.20.5

kikugie commented 1 month ago

Ok, it was less work than I reanticipated

sternschnaube commented 1 month ago

I'm glad to hear that 😄

kikugie commented 1 month ago

FIXED

cat-agree