fixed bug #625

[voroxity]

📝 Description

fixes bug #625

🎯 Goals

fixes bug #625

❌ Non Goals

break crafting or the oredict

🚦 Testing

Made changes, built a jar, and tried to reproduce the bug

⏮️ Backwards Compatibility

maybe. im not sure.

📚 Related Issues & Documents

Fixes #625

🖼️ Screenshots/Recordings

📖 Added to documentation?

• 📜 README.md
• 📑 Documentation
• 📓 Javadoc
• 🍕 Comments
• 🙅 No documentation needed

😄 [optional] What gif best describes this PR or how it makes you feel?

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

Walkthrough

This pull request corrects the ore dictionary name for glass components in the crafting mappings configuration file. All occurrences of the "glass" entry name have been replaced with "blockGlass" across multiple recipe definitions to align with the correct Minecraft ore dictionary standard.

Changes

Cohort / File(s)	Summary
Crafting Mappings Configuration src/main/resources/craftingmappings.json	Renamed ore dictionary entry key from "glass" to "blockGlass" across 21 recipe entries to match the correct ore dictionary name that accepts all glass types (dyed and non-dyed).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'fixed bug #625' is vague and generic, not describing what the actual fix entails.	Use a more descriptive title that explains the fix, such as 'Update glass ore dictionary name from glass to blockGlass in crafting mappings'.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description is related to the changeset as it references bug #625, which the changes address, but provides minimal detail about the actual fix.
Linked Issues check	✅ Passed	The PR changes directly address issue #625 by renaming the glass ore dictionary key from 'glass' to 'blockGlass' in craftingmappings.json, which matches the suggested fix in the issue.
Out of Scope Changes check	✅ Passed	All changes in the PR are directly scoped to fixing issue #625 - only the glass ore dictionary name is changed in craftingmappings.json without additional unrelated modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}