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

Explosives requirements infrastructure #37822

Conversation

Brian-Otten
Copy link
Contributor

Summary

SUMMARY: Infrastructure "Adds volatile_explosive requirement"

Purpose of change

The recipe for making many explosives was outdated, and not easily added to / edited.

Describe the solution

This adds a volatile_explosive requirement recipes can request which is 10g tnt equivalence of all the usable volatile explosives. We can add new explosive compounds to this group as we add more chemistry to the game. The old explosives recipe have been edited to use this new requirement.

Testing

Game loads and the recipes work as expected.

Additional context

image

Goes well with #37710 because i used the same gunpowder requirements as the old recipes, which don't match with the power of the bomb if the above PR is not merged.

Brian-Otten and others added 3 commits February 8, 2020 03:20
Accidentally used its current explosive value instead of the old gunpowder requirement.
@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling labels Feb 8, 2020
@ZhilkinSerg ZhilkinSerg merged commit 0a5669c into CleverRaven:master Feb 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Crafting / Construction / Recipes Includes: Uncrafting / Disassembling [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants