Skip to content

Recursion in horizon/scoreboard#1

Closed
Kaperstone wants to merge 1 commit intoBlueMountainsIO:masterfrom
Kaperstone:master
Closed

Recursion in horizon/scoreboard#1
Kaperstone wants to merge 1 commit intoBlueMountainsIO:masterfrom
Kaperstone:master

Conversation

@Kaperstone
Copy link
Copy Markdown

When the server starts up, it adds a remote event named scoreboard:update and then it calls the same event, essentially re-calling itself, when the player presses tab it will only add a recursion to the already running recursions.

This commit fixes this, by splitting the event to two pieces, one that gets the all the online player data on the server side and the second that does the rest on the client side.

When the server starts up, it adds a remote event named `scoreboard:update` and then it calls the same event, essentially re-calling itself, when the player presses `tab` it will only add a recursion to the already running recursions
@Kaperstone Kaperstone changed the title 🐛 Fixing recursion Recursion in horizon/scoreboard Aug 21, 2019
@BlueMountainsIO
Copy link
Copy Markdown
Owner

It works this way:

  1. Client calls scoreboard:update on the server.
  2. Server calls scoreboard:update on the client with the new data.

@Kaperstone Kaperstone closed this Aug 23, 2019
BlueMountainsIO pushed a commit that referenced this pull request Dec 12, 2019
Improve AR aim left hand positions
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