Skip to content

Fix enchanted book requests 1.21#10791

Merged
someaddons merged 7 commits intoversion/1.21from
compatmanagersync
Apr 21, 2025
Merged

Fix enchanted book requests 1.21#10791
someaddons merged 7 commits intoversion/1.21from
compatmanagersync

Conversation

@someaddons
Copy link
Copy Markdown
Contributor

Closes #
Closes #

Changes proposed in this pull request

  • Fix enchanted book requests on postbox
  • CompatibilityManager is now side specific, due to Itemstacks now being side sensitive
  • Fix enchanted books now showing up on sync due to item comparisons failing as CHECKED_NBT_KEYS were loaded too late
  • Switched compat manager sync to player login, as it has to happen after registries got loaded

Testing

  • [ x] Yes I tested this before submitting it.
  • [x ] I also did a multiplayer test.

Review please

Compatmanager is now side specific, due to Itemstacks now being side sensitive
Fix enchanted books now showing up on sync due to item comparisons failing as CHECKED_NBT_KEYS were loaded too late
Switched compat manager sync to player login, as it has to happen after registries got loaded
@someaddons someaddons changed the title Compatmanagersync Fix enchanted book requests 1.21 Apr 13, 2025
@uecasm
Copy link
Copy Markdown
Contributor

uecasm commented Apr 14, 2025

If ItemStacks are side-sensitive, there might need to be changes to FurnaceRecipes as well. Currently this is populated on server side and synced to client only if not single-player.

@someaddons
Copy link
Copy Markdown
Contributor Author

good call, currently it seems to mostly affect enchanted books as their data components store a holder to a registry object but should fix the others too

@someaddons
Copy link
Copy Markdown
Contributor Author

Btw I updated this and added the furnace recipes

@someaddons someaddons merged commit 457c62b into version/1.21 Apr 21, 2025
4 checks passed
@someaddons someaddons deleted the compatmanagersync branch April 21, 2025 10:14
@uecasm uecasm mentioned this pull request May 12, 2025
2 tasks
Raycoms pushed a commit that referenced this pull request May 14, 2025
Fixes JEI recipe errors as a result of later CompatibilityManager (Fix enchanted book requests 1.21 #10791)
Fixes client-side CompatibilityManager not populating properly on /reload
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.

3 participants