Skip to content

Check if players have cells when compiling the leaderboard #52#53

Open
bradennss wants to merge 4 commits intoLuka967:masterfrom
bradennss:master
Open

Check if players have cells when compiling the leaderboard #52#53
bradennss wants to merge 4 commits intoLuka967:masterfrom
bradennss:master

Conversation

@bradennss
Copy link
Copy Markdown

@bradennss bradennss commented Dec 14, 2019

State is inconsistent, either score isn’t being updated before the leaderboard is compiled, or it’s not reactive at all to ownedCells. Spoke with Luka directly when I encountered this a few months ago, it’s likely a timing bug.

@freaky767
Copy link
Copy Markdown

I tried this pullrequest + cWS, when trying to join gives me error

[FATAL] TypeError: this.webSocket.removeAllListeners is not a function

@bradennss
Copy link
Copy Markdown
Author

@freaky767 Just remove that line, it's not needed.

@bradennss bradennss closed this Mar 1, 2020
@bradennss bradennss reopened this Mar 1, 2020
@freaky767
Copy link
Copy Markdown

using pull request; works!

Comment thread package.json
},
"dependencies": {
"uws": "10.148.1"
"@clusterws/cws": "^0.16.0"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Change ^0.16.0 to latest

@tomingrad tomingrad mentioned this pull request Dec 21, 2022
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.

3 participants