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]: Add a canEquip() method in Skills #1886

Open
2 of 3 tasks
Forixaim opened this issue Dec 3, 2024 · 0 comments
Open
2 of 3 tasks

[Suggestion]: Add a canEquip() method in Skills #1886

Forixaim opened this issue Dec 3, 2024 · 0 comments
Labels
enhancement This could be added or changed in the future.

Comments

@Forixaim
Copy link

Forixaim commented Dec 3, 2024

Have you checked if there's any similar suggestions ?

  • I checked, there are no similar suggestions that have been reported.

Are you using the latest Epic Fight ?

  • I checked I'm using latest Epic Fight version.

Is this suggestion related to another mod? (i.e: add animations for X mod)

  • This is related to other mods.

The mod

Not Applicable

Minecraft Version

1.20.1

What's your suggestion?

This would be a quite simple change I think. When you're changing skills in the game, there can be a method that checks for some condition to fulfill when trying to equip a skill.

For example

You want to equip Skill A but Skill A requires Skill B and C to be equipped, and when you have them equipped you can equip said skill. But when you remove Skill B or C, Skill A will be removed alongside it.

I believe this allows the creation of very unique skills that require other skills to be able to use. I would myself keep the canEquip method pretty minimal so other programmers can tweak their own skills to fit what they like best.

@Forixaim Forixaim added the enhancement This could be added or changed in the future. label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This could be added or changed in the future.
Projects
None yet
Development

No branches or pull requests

1 participant