Skip to content

fix: croptopia salt ore drops ATO salt - #55

Merged
JaronZ merged 1 commit into
mainfrom
fix-croptopia-salt-loot-table
Aug 12, 2025
Merged

fix: croptopia salt ore drops ATO salt#55
JaronZ merged 1 commit into
mainfrom
fix-croptopia-salt-loot-table

Conversation

@JaronZ

@JaronZ JaronZ commented Aug 12, 2025

Copy link
Copy Markdown
Member

No description provided.

@JaronZ JaronZ added this to the Forgeworks 1.4.1 milestone Aug 12, 2025
@JaronZ JaronZ self-assigned this Aug 12, 2025
Copilot AI review requested due to automatic review settings August 12, 2025 10:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the loot table for Croptopia's salt ore block to drop the correct salt item from the All The Ores (ATO) mod instead of the default Croptopia salt.

  • Updates the salt ore loot table to drop "alltheores:salt" instead of the implicit Croptopia salt
  • Maintains proper silk touch behavior (drops the ore block itself when mined with silk touch)
  • Preserves fortune enchantment mechanics with 2-5 salt drops and bonus multiplier

@JaronZ
JaronZ merged commit 8a62026 into main Aug 12, 2025
3 checks passed
@JaronZ
JaronZ deleted the fix-croptopia-salt-loot-table branch August 12, 2025 10:27
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.

2 participants