Botania (Fabric/Quilt)

Botania (Fabric/Quilt)

6M Downloads

[Suggestion] Utilise Common Tags

xanthian opened this issue ยท 4 comments

commented

Mod Loader

Fabric

Minecraft Version

1.20.1

Botania version

1.20.1-441

Modloader version

Quilt 0.22

Modpack info

Stitched - https://modrinth.com/modpack/stitched

The latest.log file

https://modrinth.com/modpack/stitched

Issue description

Consider utilising common tags in recipes.
For example.

Crafty Crate - Use c:workbench in place of minecraft:crafting_table
Currently only the MC default crafting_table is usable, there are many mods that add other crafting_tables.

Steps to reproduce

No response

Other information

No response

commented

While conventional tags are indeed a Fabric interface, fewer of those seem to actually be standardized than you'd think. (e.g. c:workbench isn't mentioned in https://github.com/FabricMC/fabric/tree/1.20.3/fabric-convention-tags-v1/src/generated/resources/data/c/tags/items)
Recipes are modifiable statically via data packs or dynamically via mods like KubeJS, which both are ways pack authors can use to adapt individual mods' recipes to better fit the combination of mods in the pack.
(But I do agree that Botania should probably start using the existing convention tags, even though that will make it necessary to define certain recipes separately for Forge and Fabric.)

commented

We'll probably look into this in 1.21 with the tag unification: neoforged/NeoForge#135 / FabricMC/fabric#3310

commented

tag unification for 1.21 modloaders will have c:player_workstations/crafting_tables now

Added cause of this use case being linked to the unification prs lol

commented

Also, surprised that Crafty Craft considers c:workbench to contain Crafting table when i assumed people saw that tag for villager job site blocks. At least in tag rework, it'll be much more clear of what belongs where