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

Add mud bricks to paths and sanity-fix the dyer #9894

Merged
merged 5 commits into from
Jul 28, 2024

Conversation

uecasm
Copy link
Contributor

@uecasm uecasm commented Apr 5, 2024

Closes discord request
Closes discord request
Closes discord request

Changes proposed in this pull request:

  • Port Add mud bricks to paths #9288
  • Move a bunch of DO ingredients from mechanic to sawmill or stonemason, to be more consistent with regular recipes.
  • Fix mechanic sometimes duplicating DO ingredients handled by other crafters.
  • Teach the dyer to obey guild rules
    • Remove various duplicated crafting items already handled by other crafters (e.g. beds from wool, glass panes from glass).
    • They can still craft those items, but only with the recipes that use dye.
  • Allow postbox requests for dyed leather armor (if you have an example in your inventory).
  • Show example dyed leather armor in the dyer's JEI info.
    image
  • Add path block auditor. (This was already visible in the block tags audit, but it's easier to read this way.)

[x] Yes I tested this before submitting it.
[ ] I also did a multiplayer test.

Review please

Raycoms
Raycoms previously approved these changes Apr 6, 2024
@someaddons
Copy link
Contributor

while on this we probably should also add the missing cracked variants for e.g. stone brick

@@ -28,6 +28,9 @@
"minecraft:chiseled_nether_bricks",
"minecraft:gravel",
"minecraft:dirt_path",
"minecraft:mud_bricks",
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you already add the missing strong brick, stone brick stairs, cracked etc?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cracked stone bricks are already in there via the STONE_BRICKS tag.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've added a path tag audit now, which makes it easier to see which blocks are paths and which aren't.

@uecasm uecasm changed the title Add mud bricks to paths Add mud bricks to paths and sanity-fix the dyer Jul 27, 2024
@Raycoms Raycoms merged commit ef4f68e into version/main Jul 28, 2024
2 checks passed
@Raycoms Raycoms deleted the fix/muddy-paths-120 branch July 28, 2024 12:16
Raycoms added a commit that referenced this pull request Aug 4, 2024
Port Add mud bricks to paths #9288
Move a bunch of DO ingredients from mechanic to sawmill or stonemason, to be more consistent with regular recipes.
Fix mechanic sometimes duplicating DO ingredients handled by other crafters.
Teach the dyer to obey guild rules
Remove various duplicated crafting items already handled by other crafters (e.g. beds from wool, glass panes from glass).
They can still craft those items, but only with the recipes that use dye.
Allow postbox requests for dyed leather armor (if you have an example in your inventory).
Show example dyed leather armor in the dyer's JEI info.
image
Add path block auditor. (This was already visible in the block tags audit, but it's easier to read this way.)
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