Replace copper ingot with copper from original Minecraft.
Description
on 1.17, the copper was added, however, slimefun didnt removed it yet. causing the copper ingot(sf) still remaining. this commit replaces Copper Ingot(SF) in favor of Copper ingot from vanilla minecraft.
all users making a new addon should replace all usage of SlimefunItems.COPPER_INGOT to new ItemStack(Material.COPPER_INGOT).
Proposed changes
Remove SlimefunItems.COPPER_INGOT and replace with new ItemStack(Material.COPPER_INGOT)
Related Issues (if applicable)
Checklist
- [ ] I have fully tested the proposed changes and promise that they will not break everything into chaos.
- [ ] I have also tested the proposed changes in combination with various popular addons and can confirm my changes do not break them.
- [ ] I have made sure that the proposed changes do not break compatibility across the supported Minecraft versions (1.16.* - 1.20.*).
- [ ] I followed the existing code standards and didn't mess up the formatting.
- [ ] I did my best to add documentation to any public classes or methods I added.
- [ ] I have added
NonnullandNullableannotations to my methods to indicate their behaviour for null values - [ ] I added sufficient Unit Tests to cover my code.
Pro Tip! You can help us label your Pull Requests by using the following branch naming convention next time you create a pull request. โค๏ธ
| Branch naming convention | Label |
|---|---|
feature/** |
๐ Feature |
fix/** |
โจ Fix |
chore/** |
๐งน Chores |
api/** |
๐ง API |
performance/** |
๐ก Performance Optimization |
compatibility/** |
๐ค Compatibility |
If your changes do not fall into any of these categories, don't worry. You can just ignore this message in that case! ๐
I think this would never be considered before the recipe system rewrite (support interchangeable ingredients), but we can see what admins think.
agreed ybw. there's too much impact that wasn't considered as it will affect every addon and break them all, along with every players base, sorting systems, network systems, etc., much more care has to be put into this before it would be rolled out
D'oh, i didnt reailze that. ๐ต
But you should consider that Copper was Added in 1.17.
but the copper from slimefun was really like existed in 1.17~1.21. Why do you even want to keep it? They are @Deprecated. It Must be replaced with new Itemstack(Material.COPPER_INGOT).
If Slimefun copper is there, whats the point for Copper in Minecraft?
For that, i think the old SlimefunItems.COPPER_INGOT Should be removed,
And Must be replaced to new ItemStack(Material.COPPER_INGOT).
Let's ship it on Slimefun 5!
Let's ship it on Slimefun 5!
shush! no saying about slimefun 5. this is slimefun 4.