search

Alexthw46 / Ars-Elemental

Add-on to Ars Nouveau, based on elemental stuff
https://www.curseforge.com/minecraft/mc-mods/ars-elemental-elemental-spell-foci
GNU General Public License v3.0
4 stars 7 forks source link

Add missing Thread Tied on Elemental Armor #8

Closed ffuentesm closed 1 year ago

ffuentesm commented 1 year ago

Here we have two armors with the Tier 2 image image As you can see, only the Base Ars armor has the Tier tag on the description.

Alexthw46 commented 1 year ago

Elemental armors are always of tier3. They can be crafted from any tier of the corresponding arcanist piece, so displaying a tier that is always the same would be pretty useless, they can't be upgraded further.

ffuentesm commented 1 year ago

Would have been nice to know that before. Even if "useless" it would keep the consistency of things. Also, would be nice to block the "armor upgrade" from working on the Elemental armor. Because the Armor upgrade crafting still accepts the elemental armor even though they are already maxed out.

ffuentesm commented 1 year ago

Actually no, I was able to Upgrade the Geomancer's set to Tier 2 image image So that is making me confused

Alexthw46 commented 1 year ago

Apparently the armor needs to be equipped at least once for the tier to update to 3. I'll try moving the check to the inventory tick instead of the equipped and probably lock the elemental upgrade to tier3 armors so they don't risk remaining in the fake tier 1 stage.