Skip to content

[Request] Tile Entities with NBT in recipes #31

Description

@aaronhowser1

I saw this:

{
    "type": "compactcrafting:block",
    "block": "minecraft:block_id_here",
    "properties": {
        "{property}": ["accepted", "values", "here"]
    }
}

but im not really sure what it means by property. I tried this:

            "P": {
              "type": "compactcrafting:block",
              "block": "botania:mana_pool",
              "properties": {
                "mana": 1000000
              }
            }

but it works with mana pools that don't have any mana in them. Replacing "mana" with "{mana}" also doesn't work.

Activity

  1. SizableShrimp commented on Jun 22, 2021

    @SizableShrimp
    Contributor

    They're blockstate properties. You need to use the property ids and their allowed values. If you're having a hard time making these by hand, you can use a dev build with helper commands to do so. See the Compact Mods discord in the README. But also you should be able to see the right names for the blockstate properties in F3 when looking at the block.

  2. aaronhowser1 commented on Jun 22, 2021

    @aaronhowser1
    Author

    So, no there isn't a way to use NBT? Just blockstates?

  3. SizableShrimp commented on Jun 22, 2021

    @SizableShrimp
    Contributor

    I don't think blocks can hold NBT, do you mean tile entities?

  4. aaronhowser1 commented on Jun 22, 2021

    @aaronhowser1
    Author

    Yes, sorry that's what I meant. I forgot those weren't technically the same thing.

    My example is trying to combine 27 full mana pools, though.

  5. SizableShrimp commented on Jun 22, 2021

    @SizableShrimp
    Contributor

    Matching tile entities nor their NBT isn't supported at the moment. So, no, you can't do that.

  6. changed the title [-]Is it possible to require blocks have a specific NBT?[/-] [+][Request] Tile Entities with NBT in recipes[/+] on Jun 22, 2021
  7. robotgryphon commented on Jun 22, 2021

    @robotgryphon
    Collaborator

    So, I would like to support this (likely with the data name for the JSON group) but unfortunately due to how CC currently scans on block updates, this is tricky.

    I'll probably be deferring this until proxy blocks are done, so you can do manual recipe scans with redstone signals. The actual data can be matched by dumping tile NBT via save on the server side.

  8. self-assigned this
    on Jun 22, 2021
  9. locked and limited conversation to collaborators on Aug 20, 2022
  10. converted this issue into a discussion #63 on Aug 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions