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

Phrasing in Tinker's Anvil description #2521

Closed
g0urra opened this issue Jun 24, 2021 · 7 comments
Closed

Phrasing in Tinker's Anvil description #2521

g0urra opened this issue Jun 24, 2021 · 7 comments

Comments

@g0urra
Copy link
Contributor

g0urra commented Jun 24, 2021

Description
Change the description of Tinker's Anvil to "Craftable with almost any alloyed metal block."

Why would you like this added/changed?
The current description is "Craftable with almost any metal block", which is misleading as the amount of metal blocks that can't be used (24) to make the Anvil is more than those that can (15). Changing it to include "alloyed" would be more accurate.

@MVethana
Copy link
Contributor

From Splutty#4917 testing on a clean server with just TC installed these are the materials that did end up working so it looks like its a Tinkers issue and not us

Slimesteel, Tinker's Bronze, Rose Gold, Pig Iron, Queen's Slime, Manyullyn, Hepatizon, Netherite, Bronze, Brass, Electrum, Invar, Constantan, Pewter, Steel

@KnightMiner
Copy link

KnightMiner commented Jun 24, 2021

That is because those are the materials that are supposed to work. You are free to add more using the tag, but its intentionally only alloys.

This bug is entirely on the pack, as you added that incorrect tooltip

@Splutty
Copy link

Splutty commented Jun 24, 2021

Yeah. Small communication error. The actual text is from the modpack, the list of anvils is from 'clean'. Sorry.

@NielsPilgaard
Copy link
Collaborator

Hmm, does it accept all alloys in E6 atm?

@KnightMiner
Copy link

If not, add the alloy's tag to the item tag tconstruct:anvil_metal. Tinkers adds a bunch based on the metals we natively support, but that list was mostly copied from Immersive Engineerings list

@MuteTiefling
Copy link
Contributor

Cool, I'll just add to that tag then instead of what I was doing :D

@Splutty
Copy link

Splutty commented Jun 24, 2021

A happy ending after all!

MuteTiefling added a commit to MuteTiefling/Enigmatica6 that referenced this issue Jun 24, 2021
Resolves EnigmaticaModpacks#2521

Changes the tooltip to specify all alloy blocks may be used. Expands the `tconstruct:anvil_metal` tag to include all other alloys in the pack.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants