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

[Suggestion]: Enchantment Beacon Upgrade #178

Open
BenGoldberg1 opened this issue Apr 5, 2023 · 2 comments
Open

[Suggestion]: Enchantment Beacon Upgrade #178

BenGoldberg1 opened this issue Apr 5, 2023 · 2 comments
Labels
'Back Burnered' Known issues, but not important enough to take away from pushing updates enhancement New feature or request [1.19]

Comments

@BenGoldberg1
Copy link

BenGoldberg1 commented Apr 5, 2023

A Pedestal with the suggested Upgrade can hold any Enchanted Book.

If any other Pedestal linked with this Pedestal uses a tool or weapon, and that item is enchantable but unenchanted, then the enchantments from the enchanted book are copied to the item before it is used, and deleted from the item after it is used. The enchanted book would not be consumed.

Requiring XP for each operation might be a good default.

The Enchantment Beacon Upgrade might be crafted from an Upgrade Base and any Enchanted Book.

Some configurable limits on how many Enchantment Beacon Pedestals any given tool/weapon using Pedestal may be linked (and vice versa) to might be a good idea.

@Mowmaster Mowmaster added enhancement New feature or request [1.19] labels Apr 5, 2023
@Mowmaster
Copy link
Owner

A pedestals work card to target pedestals would be ideal first of all.
Secondly i'd have to implement some sort of 'effect tool' slot that can be in every pedestal, and then somehow have this upgrade send 'updates' to refresh the 'effect tool' it could be done.
Using multiple pedestals to effect things sounds nice, but ideally just enchant a book with whatever enchants you want and then ill charge an 'xp cost' as fuel to run it.
this would either be a running cost, or a per pedestal cost to reapply the effect (configurable)

@Mowmaster
Copy link
Owner

the only use case i can think of is when you want to scale up a setup, but if you can come up with a few scenarios where this could be useful, ill consider this addition.

@Mowmaster Mowmaster added the 'Back Burnered' Known issues, but not important enough to take away from pushing updates label May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
'Back Burnered' Known issues, but not important enough to take away from pushing updates enhancement New feature or request [1.19]
Projects
None yet
Development

No branches or pull requests

2 participants