-
Notifications
You must be signed in to change notification settings - Fork 67
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
Some adjustments to crafts might not be future-compatible #192
Comments
as i'm writing this, i'm half asleep, but i fear i'll forget it in the morning if i don't reply now. my PR against moreblocks removed some recipe changes that were forced if you used moreblocks. such changes were amongst the most minor features provided, and i deemed such changes to be more fitting to a specific server's overrides, vs. trying to be opinionated about balancing how many items should result from certain crafting recipes. if you want a separate mod that'll make everything "exactly like it was before", i could do that, but every server operator i've spoken to already has custom mods to balance and integrate the minetest mod ecosystem. |
to be more precise, sure, i'll create a mod that overrides the recipes the way they were overridden before, but it'll be a completely separate mod. |
I think this is the best course of action too. Ideally, this mod should be available before More Blocks 3.0.0 is formally released, so that I can link to it in the release notes. |
I added a default-off option to enable those old recipes: |
Awesome! To be completely fair, these crafting adjustments also broke another WhyNot rule, in that they could be seen as "taking over the game." @fluxionary 's PR would fix that issue in the context of WhyNot, and if we take advantage that it is in a seperate mod (which can be disabled or simply excluded) then that also fixes the other problem. |
Linking afore-mentioned pr to link all the things: #191 |
Hey there, we've been reviewing your mod in this issue in the WhyNot project: minetest-whynot/whynot-game#25
There's a few concerns, but I'll just post em one-at-a-time so you, the developer of this project, can choose to accept or reject any of these suggestions on their own. They might not come in very fast. The three of us at WhyNot are somewhat busy.
And yes, I'd be willing to send you a PR with a solution for any of these issues.
These crafts are the ones I'm concerned about. To quote myself from the issue I linked,
The concern is mostly that if another mod chooses to make a modification to these recipes like moreblocks does, it might not result in the intended value. This could also be problematic if the mod changes their recipe output.
The text was updated successfully, but these errors were encountered: