-
-
Notifications
You must be signed in to change notification settings - Fork 58
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
Implement custom crafting recipes as official JSON recipes #1053
Conversation
Leaving out: auto crafter, heart canister tier 2, relocators, scuba gear Slightly modifiers: ender hopper minecart, liquid minecart Discarding item data: boots of ostara, mountaineering
I think @Bloo-dev should weigh in on this convo, but I think porting the forming press recipes into normal crafting kinda bypasses the extra step that makes those double brick recipes warranted to be in the game. Perhaps we can switch it to just taking item stacks and using those (e.g. 8 cobblestone in one slot + 1 clay would create stone bricks). The original CC system wouldn't allow for differently stacked item counts, but given that system will eventually be gone we can rewrite Forming Press (again :P) |
When guidebook added the I also wish we didn't have to duplicate custom item definitions, once in the loot table (which is a nice /give alternative) and once in the recipe. I think it might be good to either generate/mutate the recipe files using the loot table definitions as a source of truth, or generate the loot table files from these recipe definitions as a source of truth. On a side note, is smithed crafter mostly dead with componentized output recipes? Is smithed crafter cross-features still something we think would be good? |
I believe the ideal situation is to get rid of
We could probably do something similar to CMD where we inject custom data into the
I think ideally the Smithed Crafter is phased out just like the GM4 custom crafter. Given we have had hints of custom items, I don't think the smithed crafter should be a priority. The smithed libraries are pretty neglected, but rx has been wanting to do a "weld plugin" which would take all data driven recipe files and add them to the crafter. So if that ever becomes a thing (unlikely to happen before custom items tbh), we'd just need our recipe files ready. |
…into json-recipes
I think this PR is ready now. I did not switch This PR is meant as an initial step towards more standardized items and recipes, to make these recipes more accessible to players. It can definitely be improved in the future. |
…into json-recipes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the automatic overlay stuff you've got here Misode 💙
…1053) * Port custom crafting recipes to official JSON recipes when possible Leaving out: auto crafter, heart canister tier 2, relocators, scuba gear Slightly modifiers: ender hopper minecart, liquid minecart Discarding item data: boots of ostara, mountaineering * Add recipe unlock advancements * Update record crafting advancement * Optimize scuba gear advancement * Fix red_nether_bricks unlock was missing * Add group field to forming press recipes * Add standard crafting recipes except stairs and slabs * Generate standard crafting slabs and stairs recipes * Group stairs and slabs recipes together with vanilla recipes * Revert forming press changes * Update recipes to 1.21.3 * Update standard crafting recipes * Remove attribute prefix from crampons recipe * Keep the old format for gm4_recipes * Backport recipes with beet * Handle when both attributes and recipe rewrites affect the same file * Catch recipe rewrite errors * Remove manually backported soul glass recipe
Notes
Todo
group
field for forming press recipesFuture work
custom_name
toitem_name
andrarity
[GM4] <module name>
tag line