stujones11 / minetest-3d_armor

Visible player armor & wielded items for minetest
Other
56 stars 98 forks source link

Settingtypes #101

Closed tacotexmex closed 7 years ago

tacotexmex commented 7 years ago

Global setting types for major configuration variables would be very useful, enabling server admins to configure the mod and be able to being able to pull the latest mod version.

stujones11 commented 7 years ago

I am not sure I understand, all the settings are global already in armor.config. The old global configuration variables are still supported for now, however, this will be eventually be removed so I'd recommend that you update to the new configuration system as soon as possible.

tacotexmex commented 7 years ago

I am sorry. I did not read the documentation well enough. Thank you for replying, regardless. I shall repent!

tacotexmex commented 7 years ago

Hey, I just tried

armor_material_wood = false armor_material_cactus = false armor_material_steel = false armor_material_bronze = false armor_material_diamond = false armor_material_gold = false armor_material_mithril = false armor_material_crystal = false

in order to get only wieldview to work (in conjunction with playeranim). But the armor types still shows i craftguide and are still craftable. Seems like a bug?

stujones11 commented 7 years ago

You are right, it was a bug but this should be fixed now 1e059f2557ef28a8a93d81e5fe3b694c8c3d261c Thank you for reporting this, please re-close the issue when you are happy it is working.

stujones11 commented 7 years ago

I take it from the emote that it worked, otherwise just re-open it again :)

tacotexmex commented 7 years ago

I haven't checked yet but will eventually!

tacotexmex commented 7 years ago

Works fine. It's good that it's possible to turn everything off but wieldview. For now that is all I need.