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

Rework master.ts #198

Merged
merged 9 commits into from
Nov 18, 2023
Merged

Rework master.ts #198

merged 9 commits into from
Nov 18, 2023

Conversation

OmniTroid
Copy link
Contributor

@OmniTroid OmniTroid commented Nov 18, 2023

Use AOServer as a unified type, rewrite the logic around this and remove some unused variables.
Add playercount in parentheses in server names.

@OmniTroid
Copy link
Contributor Author

image
This is what new playercount looks like

@stonedDiscord
Copy link
Member

sneaky wss

@stonedDiscord stonedDiscord merged commit 21a2c48 into AttorneyOnline:master Nov 18, 2023
1 check passed
@OmniTroid OmniTroid deleted the fix-https branch November 18, 2023 18:18
@OmniTroid
Copy link
Contributor Author

That reminds me that there's a bug here: if a server does have wss enabled, joining it will probably not work

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