Skip to content

Fix node digging grouops and sounds - #20

Open
Emojigit wants to merge 1 commit into
minetest-mods:masterfrom
C-C-Minetest-Server:fork-20240615-group-sound-fix
Open

Fix node digging grouops and sounds#20
Emojigit wants to merge 1 commit into
minetest-mods:masterfrom
C-C-Minetest-Server:fork-20240615-group-sound-fix

Conversation

@Emojigit

Copy link
Copy Markdown
Member

This PR fixes the digging groups and sound by having them follow their recipe node. To make sure the data is readable during registration, all material mods are added as optional dependencies.

This PR is ready for review.

Comment thread shapes.lua
is_ground_content = false,
groups = {cracky = 3, oddly_breakable_by_hand = 2, stone = 1},
sounds = default.node_sound_stone_defaults(),
groups = get_dig_groups(recipeitem),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about the stone group? I believe that one should be preserved.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stone is not a dig group, but rather a group identifying that it is stone.

@SmallJoker SmallJoker Jun 15, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know. Is there a particular reason why that group is no longer used? It can still be relevant for recipes.

Comment thread shapes.lua Outdated
This PR fixes the digging groups and sound by having them follow their
recipe node. To make sure the data is readable during registration, all
material mods are added as optional dependencies.
@Emojigit
Emojigit force-pushed the fork-20240615-group-sound-fix branch from 399f6ab to b32e979 Compare June 15, 2026 11:29
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

Successfully merging this pull request may close these issues.

3 participants