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

feat: Add soulpoint subcommand and aliases for server commands, click-to-switch server texts, fix formatting issues, command no longer hands the game #2415

Merged
merged 3 commits into from
Apr 20, 2024

Conversation

kristofbolyai
Copy link
Collaborator

@kristofbolyai kristofbolyai commented Apr 20, 2024

The timeouts were removed since the ServerListModel now automatically updates every 15 seconds, no need for the manual force updates now, and the running the command feels much faster now.

image
image
image

…-to-switch server texts, fix formatting issues, command no longer hands the game

The timeouts were removed since the ServerListModel now automatically updates every 15 seconds, no need for the manual force updates now, and the running the command feels much faster now.
@kristofbolyai kristofbolyai changed the title feat: Add soulpoint subcommand and aliases for server commands, click… feat: Add soulpoint subcommand and aliases for server commands, click-to-switch server texts, fix formatting issues, command no longer hands the game Apr 20, 2024
Copy link
Contributor

@ShadowCat117 ShadowCat117 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, one thing I will note if you want to fix it here there's been a bug for a while if you just run /servers info without an argument it sends you back to the lobby, I'm not quite sure what causes it

@kristofbolyai
Copy link
Collaborator Author

Looks good, one thing I will note if you want to fix it here there's been a bug for a while if you just run /servers info without an argument it sends you back to the lobby, I'm not quite sure what causes it

/servers is a Wynn command. So that, I guess.

@ShadowCat117
Copy link
Contributor

/servers is a Wynn command. So that, I guess.

Ahh that would do it

@kristofbolyai
Copy link
Collaborator Author

/servers is a Wynn command. So that, I guess.

Ahh that would do it

Fixed it anyways

@kristofbolyai kristofbolyai merged commit ec06234 into fuygg_features Apr 20, 2024
1 check passed
@kristofbolyai kristofbolyai deleted the server_command_improvements branch April 20, 2024 18:19
kristofbolyai added a commit that referenced this pull request Apr 21, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request Apr 21, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
@magicus
Copy link
Member

magicus commented Apr 22, 2024

The updating every 15 seconds, were that done before? I can't figure it out from this PR. (Also, are we caching results in Athena or polling Wynncraft every 15 second?)

@kristofbolyai
Copy link
Collaborator Author

The updating every 15 seconds, were that done before? I can't figure it out from this PR. (Also, are we caching results in Athena or polling Wynncraft every 15 second?)

I recently introduced it, when I added a server uptime overlay. We use athena (for the better or for the worse, Athena performance is really bad recently..)

kristofbolyai added a commit that referenced this pull request Apr 22, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request Apr 23, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request Apr 23, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request Apr 23, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request Apr 24, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request Apr 25, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request Apr 28, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request May 5, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
kristofbolyai added a commit that referenced this pull request May 13, 2024
…-to-switch server texts, fix formatting issues, command no longer hands the game (#2415)
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.

None yet

3 participants