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

block entities servux paste #3

Conversation

zly2006
Copy link

@zly2006 zly2006 commented Jun 28, 2024

No description provided.

@sakura-ryoko
Copy link
Owner

Let me know if you think this is good enough for review.

@zly2006
Copy link
Author

zly2006 commented Jun 29, 2024

I did not change a lot on litematica, currently this works well. review this please, and servux as well

@sakura-ryoko
Copy link
Owner

I found a Servux side issue for this with the Packet Splitter. It's not a "bug" but it's more of a "we need to allow Packet Splitter to receive larger slices from a Client" issue.

@sakura-ryoko
Copy link
Owner

All set for merge

@sakura-ryoko sakura-ryoko merged commit 6c82dee into sakura-ryoko:pre-rewrite/fabric/1.21-block_entities Jun 29, 2024
2 checks passed
@zly2006 zly2006 deleted the pre-rewrite/fabric/1.21-block_entities-servux-paste branch June 29, 2024 15:52
sakura-ryoko added a commit that referenced this pull request Jul 3, 2024
* Bootstrap for entity data

* Fix data channel for servux

* block entities servux paste (#3)

* feat: paste to servux

* feat: paste to servux

* feat: paste to servux

* fix

* Forgot to merge my fixes

* Forgot to merge my fixes

* Fix Accurate Placement for Crafter Orientations and a few other Block States

* Update accuratePlacement Protocol V3 for use with Servux

* Fix:  Item Frame floor / ceiling Yaw values.

* Add settings for Entity Data Syncer

* Add settings for Entity Data Syncer

* code cleanup / Litematic Entity NBT Save handling

* code cleanup / Litematic Entity NBT Save handling

---------

Co-authored-by: Liyan Zhao <[email protected]>
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