Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Armstuff reciepes useful again. And get rid of unnecessary sheet reciepes #810

Merged
merged 4 commits into from
Sep 5, 2021

Conversation

Lamandus
Copy link
Contributor

@Lamandus Lamandus commented Aug 25, 2021

Summary

SUMMARY: Content "Armstuff makes sense again and get rid of unnecessary sheet reciepes"

Purpose of change

Getting rid of half baked stuff is our game. So I got rid of the very unbalanced arm stuff reciepes. They don't take days to craft and use materials, which are used for other items as well. I got most of the values of old DDA .C build.

Describe the solution

stuff I did

Describe alternatives you've considered

going to sleep

Testing

works

Additional context

Lamandus added 2 commits August 25, 2021 23:44
Crafting arm stuff now is back in line with other reciepes in Time and material
hush hush
@chaosvolt
Copy link
Member

Nice. One side note, it may still be useful to retain the frame loom and repurpose it towards being actually useful. Basically, my idea I'd had kicked onto the to-do list was to add alternative recipes for rags, and possibly also sheets, allowing players to use a loom for a faster craft time.

@Coolthulhu Coolthulhu self-assigned this Aug 27, 2021
@Coolthulhu
Copy link
Member

Recipes must be obsoleted before removal, otherwise you get debugmsgs on load.
Check out data/json/recipes/recipe_obsolete.json. On a side note, it would be better if it was in data/json/obsoletion with the rest of the obsolete stuff - when the time comes, the directory will be purged.

@Coolthulhu
Copy link
Member

Has a simple conflict with an earlier PR - the one that obsoleted hand axes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants