Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@peppy
Copy link
Member

@peppy peppy commented Aug 26, 2025

Noticed while investigating #34641. Likely doesn't resolve that issue, but does resolve another.

The sort in 3f179e3 just feels right to have there. Avoids some edge case where it takes 1 second to show correct state on initial display.

peppy added 2 commits August 26, 2025 17:51
… wrong order

Due to `Perform` being used from a BDL method in conjunction with
`Success` (which is scheduled to the *update* thread), there was a
chance that the order of execution would be not quite as intended.

To rectify, let's not use `Success` and just continue with synchronous
flow.
@peppy peppy requested a review from bdach August 26, 2025 08:53
@peppy peppy added the area:online functionality Deals with online fetching / sending but don't change much on a surface UI level. label Aug 26, 2025
@bdach bdach merged commit 8f628d1 into ppy:master Aug 26, 2025
5 of 9 checks passed
@peppy peppy deleted the fix-daily-challenge-leaderboard-skip branch August 27, 2025 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:online functionality Deals with online fetching / sending but don't change much on a surface UI level. size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants