Epic Fight

Epic Fight

13M Downloads

Recomendation: adding a tag for shields?

Shibva opened this issue · 9 comments

commented

im not 100% sure about this but there is this addon called epicfightNBT that allows for tetra tools to be assigned values to allow for it to have an overall makeup of the took based on its upgrandes, but alas, the shields dont have a value and when using them it dosent render properly like with the combat shield animation. idk if I should be reporting this here or the other, but I think maybe something can be done cus I hope that I can use custom shields from tetra with them without the animation breaking

commented

Is there a demo of this that I can test that's available?

commented

See the datapack folder in jar file. There are JSON files that are used now.

image

commented

I wonder if you can implement epic fightNBT, because what's up my brother is basically allow for tetra tools to be registered based on what part is on them or something

commented

I've made weapon type separation function by NBT value but don't know it actually works. If you have basic acknowledge about JSON, I'll give you an example I've made about tetra weapons.

commented

well turns out this addon epic fight NBT seems to be under no lisensing, so maybe that can be used as a point of refrence?

heres the link to their respitory
https://github.com/pitbox46/epicfightnbt

commented

Well, Since that addon implemented all functions regarding the problem of Tetra, It seems like I just have to add the shield category to config file.

commented

So will this addon be merged with the base code?

commented

No, It still needs if you want to add weapons like Tetra by the configuration file. Only Datapack supports NBT based separation.

commented

Well, adding a custom weapon by configuration is deprecated after 5.1.0. The latest version uses a Datapack system that also enables the players can set different kinds of weapon types based on NBT in one item.

By the way, I'll see what you mentioned about the shield tag. If it works as I thought, I'll add it to the Datapack in the next update.