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

Some adjustments to crafts might not be future-compatible #192

Open
Lazerbeak12345 opened this issue Aug 21, 2022 · 6 comments
Open

Some adjustments to crafts might not be future-compatible #192

Lazerbeak12345 opened this issue Aug 21, 2022 · 6 comments

Comments

@Lazerbeak12345
Copy link

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,

  • Changes quantity of default:sign_wall craft to old + 1
  • Changes quantity of default:paper craft to old * 4
  • Changes quantity of (re) carts:.*rail and default:.*rail craft to old + old/2

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.

@fluxionary
Copy link

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.

@fluxionary
Copy link

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.

@Calinou
Copy link
Member

Calinou commented Aug 21, 2022

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.

@fluxionary
Copy link

I added a default-off option to enable those old recipes:

5ae6dfe

@Lazerbeak12345
Copy link
Author

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.

@Lazerbeak12345
Copy link
Author

Linking afore-mentioned pr to link all the things: #191

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants