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

Merge master into 1.21.4 #871

Merged

Conversation

AlexProgrammerDE
Copy link
Contributor

This PR merges changes from master into feature/1.21.4. Notable the packets ServerboundPickItemFromBlockPacket and ServerboundPlayerLoadedPacket were now marked as ticked. ServerboundPickItemFromEntityPacket was already marked as ticked.

AlexProgrammerDE and others added 4 commits November 27, 2024 21:07
* Move reusable methods to a separate helper class

This way we allow other apps such as Geyser LocalSession to use these currently private methods without needing to copy over the code.

* Remove unused field
* Move packets to proper packages & implement game thread metadata for every packet

* Add new ticking info for 1.21.3

Notable info:
ServerboundSelectBundleItemPacket is not ticked, it's instant
ClientboundSetHeldSlotPacket is already ticked as it is just renamed ClientboundSetCarriedItemPacket

* Expose packet handler executor

* Do not tick delimiter packet

* Allow server to define packet handler executor factory

* Update protocol/src/main/java/org/geysermc/mcprotocollib/network/packet/Packet.java

Co-authored-by: chris <[email protected]>

---------

Co-authored-by: chris <[email protected]>
@onebeastchris onebeastchris merged commit 8806d1e into GeyserMC:feature/1.21.4 Dec 5, 2024
1 check passed
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.

2 participants