Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Sweeping edge enchantment name visibility #1741

Closed
DUDEbehindDUDE opened this issue Dec 26, 2020 · 6 comments · Fixed by #3615
Closed

Sweeping edge enchantment name visibility #1741

DUDEbehindDUDE opened this issue Dec 26, 2020 · 6 comments · Fixed by #3615
Labels
Feature Request A feature request.

Comments

@DUDEbehindDUDE
Copy link

What feature do you want?
A description for the sweeping edge enchantment to be visible when it's on an item; since it's a Java only enchant it doesn't show up on a bedrock client besides in the current enchanting table gui

Alternatives?
I don't know of any

@Camotoy Camotoy added the Feature Request A feature request. label Dec 26, 2020
@Camotoy
Copy link
Member

Camotoy commented Dec 26, 2020

Related to #1409.

@ImDaBigBoss
Copy link
Contributor

I think this is handled by GeyserOptionalPack now isn't it?

@Camotoy
Copy link
Member

Camotoy commented Jun 3, 2021

No. This issue requests for the description and not the particle.

@Camotoy Camotoy changed the title Sweeping edge Sweeping edge enchantment name visibility Jul 3, 2021
@fabrematic
Copy link

It seems that Geyser is dropping sweeping edge information when translating enchantments, making it impossible for a client-side addon to show this Java-only enchantment. Perhaps it is possible to add a geyser-only nbt entry for it, so it is safely ignored by standard clients, but also enables client-side recognition?

@Chew Chew mentioned this issue Dec 7, 2021
@Oofman1062
Copy link

its almost been a year since the last comment here so ill ask, will this be fixed anytime soon?

@MrBsng
Copy link

MrBsng commented Dec 15, 2022

its almost been a year since the last comment here so ill ask, will this be fixed anytime soon?

I've made an alternative @Oofman1062 @DUDEbehindDUDE . It's a spigot plugin that the server owner would need to download.
Since 7474d2c & #3615 my plugin is no longer applicable unless for some reason you're using an older version of geyser. Then you can use https://github.com/TBYT/GeyserParity/releases/tag/deprecated-release

Check out https://github.com/TBYT/GeyserParity/releases/latest if you're interest in fixing sweeping edge in the anvil!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Feature Request A feature request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants