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

Build fire ring underground #55569

Merged
merged 2 commits into from
Feb 25, 2022

Conversation

Light-Wave
Copy link
Contributor

@Light-Wave Light-Wave commented Feb 21, 2022

Summary

Bugfixes "Fire Ring can be built on underground dirt and rock floors"

Purpose of change

#54386 unintentionally made it much harder to build a fire ring in a cave. This change fixes that, and also makes it possible to build the ring on rock floors.

Describe the solution

Add a new recipe for f_firering that can be built on top of t_dirt_underground, and another that can be built on t_rock_floor.
The version built on rock floor does not need a digging implement.

Describe alternatives you've considered

Keep the digging implement requirement for rock floor, or require a pickaxe instead?
Let the fire ring be built on any fire safe terrain?

Testing

I built a fire ring underground and deconstructed it, and it seemed to work out fine. Did the same on rock floor.

Additional context

Useful for Innawood cave starts.

@Light-Wave Light-Wave changed the title Build fire ring underground [WIP] Build fire ring underground Feb 21, 2022
@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Feb 21, 2022
@Light-Wave Light-Wave changed the title [WIP] Build fire ring underground Build fire ring underground Feb 21, 2022
@curstwist curstwist added Crafting / Construction / Recipes Includes: Uncrafting / Disassembling [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions and removed json-styled JSON lint passed, label assigned by github actions labels Feb 21, 2022
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 21, 2022
@kevingranade kevingranade merged commit 944d708 into CleverRaven:master Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Crafting / Construction / Recipes Includes: Uncrafting / Disassembling [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants