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

[Blaze] Move recipes for Blaze vehicle crafting rigs from Aftershock to Blaze. #48373

Merged

Conversation

shacknetisp
Copy link
Contributor

@shacknetisp shacknetisp commented Apr 5, 2021

Summary

Bugfixes "Move recipes for Blaze vehicle crafting rigs from Aftershock to Blaze."

Purpose of change

#47279 moved vehicle parts including the metal, kitchen, and cooking crafting rigs from Aftershock to Blaze, but did not move the recipes, so they remain uncraftable unless Aftershock is also loaded.

Describe the solution

This PR moves the recipes from Aftershock to Blaze Industries, replacing with obsolete recipes in Aftershock.

Testing

Load a world with Blaze Industries and no Aftershock and gain all recipes; the MetalMaster forge buddy, KitchenMaster cooking buddy, and cooking rig will only be craftable with this PR.

@BrettDong BrettDong added Mods Issues related to mods or modding <Bugfix> This is a fix for a bug (or closes open issue) labels Apr 5, 2021
@Rivet-the-Zombie Rivet-the-Zombie merged commit 4e706d9 into CleverRaven:master Apr 6, 2021
@martinrhan
Copy link
Contributor

Those three things are missing from my game. I wasn't able to find the recipe and I wasted a lot of time to look for it. My friends also ensured this with his game and came with same result. DID YOU REALLY TESTED? I just removed the three recipes in data/mods/Aftershock/recipes/obsolete.json and the recipes are existant in game again.

@shacknetisp
Copy link
Contributor Author

I am not very familiar with the code here. If the obsolete recipes should not have been added, would you please submit a PR to fix it?
I have tested several cases back when I made this PR, but I don't remember if I tested with both Aftershock and Blaze active, so that is probably where I went wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Mods Issues related to mods or modding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants