-
Notifications
You must be signed in to change notification settings - Fork 12
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
Allowing multiple tags to be unified together #42
Comments
This feature already exists as Tag Ownerships, that was recently released :D
|
does this work with the |
It does not. And supporting the material parameter in that feature will be incredibly painful to implement. Will take a look at this next week when I find the time whether this is doable in a reasonable way. How would you imagine setting up something like this? Maybe this: "tagOwnerships": {
"c:{material}_ingots": ["c:ingots/{material}"]
} However, that may introduce edge cases since it will create that combination for each material. The tag ownership system checks if those tags actually exist. If they don't you'll get a lot of warnings. Not sure if this is a good addition since it could result in more issues than it solves. |
i see, and yeah thats basically how i would imagine it, but if the tag ownership system checks if those tags exist, would it be possible to instead create another system like "material ownerships" that would be more in line with what this feature would need to do otherwise ill just write a small kubejs script to generate the tag ownerships config automatically, though id rather do that as a last resort as that would create an extremely bloated config file |
We may introduce a custom placeholder system where you can define your own material-like lists. "placeholders": {
"materials": [
"iron",
"gold",
"copper",
"tin",
//...
],
"ownershipMaterials": [
"foo",
"bar"
]
} You could then access them dynamically in specific areas where placeholders are allowed with the usual I might need a little time to work on that though. |
yeah this makes the most since to me, would lead to the simplest config implementation no matter which standards i the dominate one, also no rush, considering both mods that this feature would majorly benefit unifying with aren't out of alpha |
Describe the feature/addition!
i would like for it to be possible to unify 2 separate tags together say
c:{material}_ingots
andc:ingots/{material}
, and it would unify them as if they were the same tag.Why would like to see this feature/addition?
on fabric we've had 2 tag standards for materials for awhile, one that the fabric port of create introduced which a lot of newer fabric mods use, (gtce unofficial, gregtech intergalactical, etc), and then the cotton standard which older mods use (modern industrialization, tech reborn, etc), and it would be nice to unify them without having to use scripts to convert tags, or to manually tag items, this could also be used in cases where mods ignore tag conventions, like unfinished ports.
The text was updated successfully, but these errors were encountered: