RayTrace082 / untamedwilds

GNU General Public License v3.0
14 stars 11 forks source link

Tetra compatibility: Any item is "Pearl+" modifier #180

Closed copygirl closed 2 years ago

copygirl commented 2 years ago

Description

When modifying a Tetra tool, any item is valid in the socket slot as a "Pearl+" modifier.

Steps To Reproduce

Comments

I looked at how tetra does things and maybe the format changed during an update and now "material": { "item": "untamedwilds:material_pearl" } would need to be changed to "material": { "items": [ "untamedwilds:material_pearl" ] }. Just a guess. I could not confirm because the way Tetra loads its datapack stuff doesn't allow me to override your additions (from what I can see, anyway). Unfortunately, this also means my modpack has this issue with no way to fix it, without removing your mod, unless I'm willing to modify and redistribute the mod's .jar file.

Also, is it just me or is the repository outdated?

Screenshot

image

Versions

RayTrace082 commented 2 years ago

The fix for this issue had been provided a few days ago by a contributor (Unkownn66) and will be released with the next version That's pretty much what happened, Tetra changed a bit the formatting in the 1.16 -> 1.18 migration, and I didn't find it out until very recently

For what it is worth, removing the tetra folder from inside the .jar's data folder will fix the issue, and I am okay with the .jar file being redistributed when it comes to bug fixing

copygirl commented 2 years ago

We can probably wait for an update. Hoping it will come out within the next week or two.

Do you mind me asking why the repository is outdated? As a modpack author I like using source code to look up certain things, when it's available.

RayTrace082 commented 2 years ago

The main reason is that I changed workspace to keep the 1.16 and 1.18 codebases separate, and I originally forgot to push the 1.18 project into its own branch, after that, it's mostly a mix of:

This should be a good reminder to publish the source code once I am finished with the update, however

RayTrace082 commented 2 years ago

Fixed