You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When merging (doesn't affect view as far as I can tell) apis with a player the production timer becomes off from the actual time by about 10 seconds. When you merge more players the timer gets off by more time.
Why
Seems like the tick_fragment value from the merged players api is replacing the accurate one from the actual server and since there is some delay in making the api call and displaying it to the user that would explain why it is off. I assume the reason it only happens with merge is because view doesn't touch as many variables or something
How
Seems like all we have to do to fix this is have merge api stop viewing the merged players tick_fragment and only use the server provided one. While we are fixing this we might as well limit a couple other things that aren't necessary like game name, settings, etc, to avoid any future problems
Note
I made this issue on the NPA repo as well since both extensions are affected by the bug
The text was updated successfully, but these errors were encountered:
What
When merging (doesn't affect view as far as I can tell) apis with a player the production timer becomes off from the actual time by about 10 seconds. When you merge more players the timer gets off by more time.
Why
Seems like the
tick_fragment
value from the merged players api is replacing the accurate one from the actual server and since there is some delay in making the api call and displaying it to the user that would explain why it is off. I assume the reason it only happens with merge is because view doesn't touch as many variables or somethingHow
Seems like all we have to do to fix this is have merge api stop viewing the merged players
tick_fragment
and only use the server provided one. While we are fixing this we might as well limit a couple other things that aren't necessary like game name, settings, etc, to avoid any future problemsNote
I made this issue on the NPA repo as well since both extensions are affected by the bug
The text was updated successfully, but these errors were encountered: