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

No support for the latest Minestom version #29

Closed
JasperTheMinecraftDev opened this issue Feb 9, 2024 · 4 comments
Closed

No support for the latest Minestom version #29

JasperTheMinecraftDev opened this issue Feb 9, 2024 · 4 comments

Comments

@JasperTheMinecraftDev
Copy link

JasperTheMinecraftDev commented Feb 9, 2024

Hey, I tried to use this on my Minestom CE server and it sadly failed to work. The Extension class isn't present in the latest Minestom versions anymore. Can you look at that?

@JasperTheMinecraftDev JasperTheMinecraftDev changed the title No support for Minestom CE No support for the latest Minestom version Feb 9, 2024
@JasperTheMinecraftDev
Copy link
Author

I've created a fork of this project which removes removes the Extension class which effectively renders it unusable as a extension but it works fine as a library. The fork can be found in my repositories if anyone needs it. Hopefully a official solution will be provided soon!

@JasperTheMinecraftDev
Copy link
Author

I've also fixed the sound errors in my fork. Is this still maintained? Otherwise I can maybe take it over.

@TogAr2
Copy link
Owner

TogAr2 commented May 2, 2024

Hey Jasper!

I finally decided to get back into this project. It is now completely updated to the latest Minestom version (along with several improvements to projectiles). I haven't done rigorous testing yet but everything seems to be working.

I looked at your fork and it seems to me that some solutions are a little hacky. I think its best to not integrate those in the original repository. You are however very welcome to contribute to this repository! I think it's great you're willing to help improving the library.

Could you maybe give me a list of all the things that you fixed in your version, so I can take a look at it? Of course you can also just open a pull request if you want. Also, what exactly are the sound errors you are talking about?

@TogAr2 TogAr2 closed this as completed May 2, 2024
@JasperTheMinecraftDev
Copy link
Author

Hey Jasper!

I finally decided to get back into this project. It is now completely updated to the latest Minestom version (along with several improvements to projectiles). I haven't done rigorous testing yet but everything seems to be working.

I looked at your fork and it seems to me that some solutions are a little hacky. I think its best to not integrate those in the original repository. You are however very welcome to contribute to this repository! I think it's great you're willing to help improving the library.

Could you maybe give me a list of all the things that you fixed in your version, so I can take a look at it? Of course you can also just open a pull request if you want. Also, what exactly are the sound errors you are talking about?

Hey!

Awesome that you decided to start working on MinestomPvP again. Can't recall the sound issue but lots of things were done, you can check it using the commits, they describe the fixed stuff. About the extension class being removed, that's because extensions have been removed from Minestom as a whole. So not really hacky :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants