search

wisp-forest / alloy-forgery

Alloy smelter mod for Fabric
MIT License
44 stars 21 forks source link

[1.19.2] kubejs fix #103

Closed quattage closed 1 year ago

quattage commented 1 year ago

this pr should contain a fix for issue 98 on 1.19.2. the build was tested in my modpack and kubejs now behaves as expected with your recipe serializer.

Noaaan commented 1 year ago

This should already be fixed as of the commits made with #98. I have already tested them, and this PR is redundant. As you see in the snippet below, it is set to air before returning if null, preventing the issue from occuring.
https://github.com/wisp-forest/alloy-forgery/blob/056c83833c2970a58f4e3415224abb5dae97a30e/src/main/java/wraith/alloyforgery/recipe/AlloyForgeRecipeSerializer.java#L98