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

Added WrapperPlayServerUnlockRecipes #382

Open
wants to merge 4 commits into
base: 2.0
Choose a base branch
from

Conversation

NoJokeFNA
Copy link
Contributor

@NoJokeFNA NoJokeFNA commented May 12, 2022

https://wiki.vg/Protocol#Unlock_Recipes (1.14 -> 1.18.2)
https://wiki.vg/index.php?title=Protocol&oldid=14299 (1.13)
https://wiki.vg/index.php?title=Protocol&oldid=12930 (1.12)
Tested Server-Versions: 1.12, 1.12.1, 1.12.2, 1.13.2, 1.14, 1.14.4, 1.15.2, 1.16.4, 1.18.2
Current issue
I have some problems with server versions 1.14 to 1.15.2 always throwing an IndexOutOfBoundsException even though nothing has changed in those versions, but I think there is a general problem with the versions and it is not the code's fault.

The Client-Version doesn't matter in this case, I've tried it with multiple ones -> same issue.

Logs
https://sourceb.in/gNedCXvso3

@NoJokeFNA NoJokeFNA changed the title feat: added WrapperPlayServerUnlockRecipes Added WrapperPlayServerUnlockRecipes May 15, 2022
@retrooper
Copy link
Owner

Interesting how you have modern and legacy recipe "inits"? I don't know if its possible to bring them into one form? If possible I don't want different fields available for different versions.

@NoJokeFNA
Copy link
Contributor Author

I know that, but I honestly don't know how we could make it different. We could use Objects, but you don't really wanna have Objects.

@NoJokeFNA NoJokeFNA force-pushed the feat/unlock-recipes-wrapper branch from 1a0b13b to 0b4fc87 Compare June 19, 2022 17:33
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