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

Partial coke & blast bricks can be placed, wiping NBT #4423

Closed
Officialy opened this issue Mar 17, 2022 · 4 comments
Closed

Partial coke & blast bricks can be placed, wiping NBT #4423

Officialy opened this issue Mar 17, 2022 · 4 comments

Comments

@Officialy
Copy link

Modpack Version

1.0.0

Describe your issue.

Upon placing a partial coke brick or blast brick, you wipe the NBT data required for the Sequenced assembly to work/continue working upon them. A solution to this would be to make it an item, with the same block model. Making it look the same, however just not placeable.

Crash Report

No response

Latest Log

No response

Have you modified the modpack?

No

User Modifications

No response

Did the issue happen in singleplayer or on a server?

Both

@Officialy Officialy added the Bug label Mar 17, 2022
@MuteTiefling
Copy link
Contributor

I'm on the fence here. Honestly, I'd rather see this setup be automated, so you're not manually moving the block around.

I agree it sucks losing that progress though.

@TemperedFool
Copy link
Collaborator

I'm wondering why placing them wipes the NBT - is that normal?

Honestly I think we should just make them items, rather than blocks - I agree we want players to automate, but there's also not really any reason for people to place them?

I think we can even tag them with create:upright_on_belt to get them to sit up on belts even. I can test later.

@MuteTiefling
Copy link
Contributor

most blocks that retain their NBT on placement/pickup are tile entities with special handling by the mod. a simple block doesn't have anywhere to store that nbt and retrieve it when it's picked up.

@TemperedFool
Copy link
Collaborator

Makes sense. I'll test swapping them to items, see if it breaks anything; I don't see why it would tbh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants