Skip to content

Conversation

@XiYang6666
Copy link

No description provided.

@jpenilla
Copy link
Owner

Do you know if this method was there since the Paper ping event was added? If not we should add some fallback logic. Also it needs to be copied in the new Paper module (with no fallback logic, since it's 1.21.8+ only). Thanks for the PR, I will pick it up later if you're no longer interested.

@XiYang6666
Copy link
Author

Thanks for the clarification! As far as I know, PaperServerListPingEvent was added in 1.12.2 and getClient().getVirtualHost() has been available ever since, so no fallback logic should be necessary.
I won’t be able to make further changes quickly due to schoolwork, so please feel free to take over the PR if needed.

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

Successfully merging this pull request may close these issues.

2 participants