-
Notifications
You must be signed in to change notification settings - Fork 53
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
tied scores rebroken #882
Comments
#418 for reference |
|
This is a side effect of 70a66e6. Scores never tracked time stamps so it was difficult to factor in the previous order of scores into the new system. In the future, order will be properly maintained by the time stamp the score was obtained. |
im pretty sure the only problem is that these old tied scores are in reverse order, its the exact same issue i reported a few months ago and it was fixed. |
70a66e6 SwitchScoreToUserIdMigration.cs L121 I believe this would be the cause. All old scores w/o a timestamp had their timestamps set to the time of when the migration occurred on the server. No real way to flip these - unless there is, however I'm not sure how we'd go about doing that. |
tied scores are supposed to be in order of time achieved so the top score should be whoever got it first. it might be in reverse order now but i can't tell
as u can see i got this score first
but now it puts me in last place instead of first
this was a issue i reported before and was fixed, don't know why it broke again
The text was updated successfully, but these errors were encountered: