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

Tried adding to my server with Ecoenchants #3

Open
pajamaben opened this issue Jul 7, 2023 · 3 comments
Open

Tried adding to my server with Ecoenchants #3

pajamaben opened this issue Jul 7, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@pajamaben
Copy link

I use EcoEnchants on my server, and when combining custom enchantment books, it throws a huge error message in the console, when adding items with the enchantments applied results in this message: "Incompatible enchantments detected for item DIAMOND_CHESTPLATE"

@lthoerner
Copy link
Owner

Hey @pajamaben, thanks for trying out BetterAnvils!

Because much of the code on this plugin relies on very static values, it is currently not compatible with custom enchantment plugins. If you would like to convert this to a feature request, we can absolutely look into it.

In terms of the error message, I realize now that I should have made a configuration option to turn off warnings like this. I will push release 0.1.2 in the next few hours where this option is disabled by default.

@lthoerner lthoerner self-assigned this Jul 7, 2023
@lthoerner lthoerner added enhancement New feature or request help wanted Extra attention is needed labels Jul 7, 2023
@pajamaben
Copy link
Author

pajamaben commented Jul 8, 2023

I figured this was the case, but from what I could see with my limited testing it did add them together on the anvil correctly, I didn't test the effects after I found the error but I'm assuming since the enchantment still showed correctly that it did work and Ecoenchantments still picked it up since since it's still on the item

@lthoerner
Copy link
Owner

That's interesting. I'm not sure what would allow EcoEnchants books to bypass the BetterAnvils anvil logic, unless they are somehow integrated with the vanilla enchantments in such a way that they are indistinguishable (books with stored Enchantments and an EnchantmentStorageMeta).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants