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

Introducing websockets - EARLY BETA, DO NOT USE THEM! #36

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

Polsulpicien
Copy link
Member

Warning

DO NOT USE THIS BRANCH!
WEBSOCKETS ARE IN EARLY BETA AND REQUIRE A SPECIAL AUTHENTICATION ON THE API
LAUNCHING THIS VERSION MIGHT RESULT IN A BAN FROM THE API

PLEASE USE THE main BRANCH INSTEAD!

Working except for the skin part, when loading players with the websocket after 3 players the skin system will crash (some internal thread issue with pyqt and the websocket, because the websocket is on another thread...), for the time being skin loading has been disabled for players loaded via the websocket.

@Polsulpicien Polsulpicien added the enhancement New feature or request label Mar 10, 2024
@Polsulpicien Polsulpicien self-assigned this Mar 10, 2024
@Polsulpicien
Copy link
Member Author

Do not merge yet (note to myself I guess?)
Need to find a solution for the skins.
Maybe a good night of sleep might help 😎

@Polsulpicien Polsulpicien mentioned this pull request Mar 10, 2024
@Polsulpicien
Copy link
Member Author

I guess it's ready to be merged. Everything is working. Missing some server side optimization before the official release though.

@Polsulpicien
Copy link
Member Author

Fixed a few small issues (didn't push yet), still doing a bit of testing then we'll be able to merge this.

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

Successfully merging this pull request may close these issues.

1 participant