Axe ring holster #36521
Axe ring holster #36521
Conversation
|
Can you give the lumberjack profession the ring holster? ❤️ |
|
Forgot about that one despite having my current character started as one. Thanks for the heads up. Added it now. |
|
Also made a mod version. |
data/json/recipes/armor/storage.json
Outdated
| "subcategory": "CSC_ARMOR_STORAGE", | ||
| "skill_used": "tailor", | ||
| "difficulty": 4, | ||
| "skills_required": [ "melee", 1 ], |
There was a problem hiding this comment.
I wonder if melee combat skill is really needed for crafting anything.
There was a problem hiding this comment.
It's what other melee holster have. Baldrics, sheaths, scabbards, and back scabbards have it so I thought I might add it as well.
There was a problem hiding this comment.
I'd prefer removing the requirement of melee skill from those recipes instead of adding it to another one.
There was a problem hiding this comment.
I'll remove that requirement on the other items if it's ok, tho it's kind of stepping outside my intended PR.
There was a problem hiding this comment.
It should be done in a different PR, it's unrelated to this one.
data/json/items/armor.json
Outdated
| "coverage": 5, | ||
| "encumbrance": 2, | ||
| "material_thickness": 1, | ||
| "use_action": { |
There was a problem hiding this comment.
This file needs a linting to get the JSON validator to accept it.
curstwist
added
[JSON]
Summary
SUMMARY: Content "Adds a holster specifically for axes"Describe the solution
Due to the lack of holster for axes and some axes not being able to be stored on some belts, I decided to make one that is similar to sheaths exclusively for axe use. Also edited axes and axe-like items to make it work with my addition
Edit 1: Also edited lumberjack profession to spawn in with one. Thanks to LaVeyanFiend for reminding me.
Testing
Checked if I can load a game, no errors. Used a debug world and spawned the materials needed to craft it, with no problems on crafting it. Tested every axe or axe-like items that I edited with the "SHEATH_AXE" flag if it can be stored, successful on all items.
Additional context
Axe ring holster samples
