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

Adding PandaSpigot to Server Software #263

Closed
Minionguyjpro opened this issue Sep 21, 2024 · 6 comments
Closed

Adding PandaSpigot to Server Software #263

Minionguyjpro opened this issue Sep 21, 2024 · 6 comments

Comments

@Minionguyjpro
Copy link
Contributor

Importance: 7/10

Problem:
I would like to use PandaSpigot along with AutoPlug, which is a well maintained fork of Paper for Minecraft 1.8.8.

Solution:
Add the feature, it shouldn't be so hard since there is an API for it available at https://downloads.hpfxd.com which seems based on OpenAPI and using GET requests stuff can be retrieved.

@Minionguyjpro
Copy link
Contributor Author

I will try looking into this, if you can give me the direction of the existing ones that use this maybe. I am pretty sure Paper or something itself also uses this API to retrieve stuff?

@Minionguyjpro
Copy link
Contributor Author

Minionguyjpro commented Sep 21, 2024

Found out the external MCServerUpdater library is responsible for this, I will try and see if I can make changes at them and then PR them into there.

EDIT: This is very easy, the API works the exact same as the one of Paper does! Am right now making changes to the MCServerUpdater repository using the fork to add PandaSpigot.

@Minionguyjpro
Copy link
Contributor Author

Minionguyjpro commented Sep 21, 2024

Changes are made and it's working! I hope it to get merged into MCServerUpdater soon. As soon as that library gets a new release, the only thing that needs to happen here is to upgrade the version of the library linked to the AutoPlug-Client project, and edit the comment of the Java code generating the updater.yml config using dyml to include the new possible value and done!

@Minionguyjpro
Copy link
Contributor Author

Minionguyjpro commented Sep 21, 2024

Great! My PR got merged. Now it's pure time to wait for the next release of their library.

@Osiris-Team
Copy link
Owner

Osiris-Team commented Sep 21, 2024

@Minionguyjpro nice while you are at it, if you are interested take a look at the other open issues over there

@Minionguyjpro
Copy link
Contributor Author

As we know this has now been integrated by #264

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