Skip to content

Conversation

HttpRafa
Copy link
Contributor

@HttpRafa HttpRafa commented Sep 1, 2025

Description

This PR implements the EnderChest block. For this functionality, it introduces the EnderChestInventory struct and a corresponding field. It also adds support for reading and writing the inventory through NBT functions. Additionally, this PR introduces the EnderChest block entity and includes other boilerplate code required for the block.

Testing

2025-09-01_19-40-57.mp4
Screenshot_20250901_194444

@HttpRafa HttpRafa marked this pull request as ready for review September 1, 2025 19:41
@4lve
Copy link
Contributor

4lve commented Sep 1, 2025

LGTM!

impl NBTStorageInit for PlayerInventory {}

#[async_trait]
impl NBTStorage for EnderChestInventory {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this should be inside the Player file

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, should I put it in a separate module in the pumpkin crate?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's already in a good place cause it's in a way tied to the player inventory

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's already in a good place cause it's in a way tied to the player inventory

I could move it under player/ender_chest_inventory.rs.

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