-
Notifications
You must be signed in to change notification settings - Fork 101
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
Protocol changes for 1.21.20 #260
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
in terms of the world, i'm pretty sure the solution is to update the blockstates nbt file that provides the runtime block map to the server |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Can you moove into discord or smth and resume here when you're done ? |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
@fBNTFeujjj @AzOxStOz @JblusItsMe Move your conversation elsewhere and it's make difficult for those who are interested in reviewing the PR |
Please don't force-push |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks very good dries-c
src/types/inventory/stackrequest/CraftRecipeAutoStackRequestAction.php
Outdated
Show resolved
Hide resolved
src/types/inventory/stackrequest/CraftRecipeStackRequestAction.php
Outdated
Show resolved
Hide resolved
src/types/inventory/stackrequest/CreativeCreateStackRequestAction.php
Outdated
Show resolved
Hide resolved
src/types/inventory/stackrequest/GrindstoneStackRequestAction.php
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CraftRecipeOptionalStackRequestAction
needs a repetitions
field
CorrectPlayerMovePredictionPacket
needs vehicleAngularVelocity
field when type is PREDICTION_TYPE_VEHICLE
@@ -50,16 +53,18 @@ public function getIngredients() : array{ return $this->ingredients; } | |||
|
|||
public static function read(PacketSerializer $in) : self{ | |||
$recipeId = $in->readRecipeNetId(); | |||
$repetitions2 = $in->getByte(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i know it's mojang fault, but it doesn't have too much sense to repetitions2 is before repetitions
$out->putBool($this->message === null); | ||
if($this->message !== null){ | ||
$out->putString($this->message); | ||
$out->putBool($skipMessage = $this->message === null && $this->filteredMessage === null); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This bool is not obvious what it is, what it does is to hide/skip the disconnection screen, please document this
final class TakeFromBundleStackRequestAction extends ItemStackRequestAction{ | ||
use GetTypeIdFromConstTrait; | ||
use TakeOrPlaceStackRequestActionTrait; | ||
enum ServerboundLoadingScreenPacketType : int{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Serverbound is a prefix for packets, i think should not be in enums
No description provided.