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

1.20.3/1.20.4 update #623

Merged
merged 31 commits into from
Dec 12, 2023
Merged

1.20.3/1.20.4 update #623

merged 31 commits into from
Dec 12, 2023

Conversation

booky10
Copy link
Collaborator

@booky10 booky10 commented Nov 17, 2023

Ofc still very much WIP, 1.20.3 doesn't even have pre-releases yet isn't released yet.

There is a major change to the protocol, components are now serialized as NBT instead of JSON.

(https://gist.github.com/booky10/e2a3682b114900f70c89c0dc12da483e has some code I've used for extracting data from vanilla)

@booky10
Copy link
Collaborator Author

booky10 commented Dec 5, 2023

I've tested most of the new stuff, works on spigot 1.20.3.

@booky10 booky10 marked this pull request as ready for review December 5, 2023 19:48
@AoElite
Copy link
Contributor

AoElite commented Dec 6, 2023

@booky10 Have you tested it on older server versions as well?

* GRASS -> SHORT_GRASS, but GRASS is deprecated.
* WrapperPlayServerTickingInfo -> WrapperPlayServerTickingState
* WrapperPlayServerTickingStep minor changes
@retrooper
Copy link
Owner

in the new NBT-Serializer for Components there's this one bug on 1.20.3/1.20.4 https://pastebin.com/mzM8TtRg

@booky10 booky10 changed the title 1.20.3 update 1.20.3/1.20.4 update Dec 8, 2023
This will now skip building nbt objects and will directly build adventure components
@booky10
Copy link
Collaborator Author

booky10 commented Dec 9, 2023

@booky10 Have you tested it on older server versions as well?

I've did some short tests (especially regarding component serialization) in 1.8.8, 1.12.2, 1.16.5, 1.20.1 and 1.20.2. Everything I've tested is working.

@retrooper retrooper merged commit 1e91905 into retrooper:2.0 Dec 12, 2023
1 check passed
@booky10 booky10 deleted the dev/1.20.3 branch December 12, 2023 21:46
@retrooper
Copy link
Owner

Thank you for the PR!

@booky10 booky10 restored the dev/1.20.3 branch May 5, 2024 17:00
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