Skip to content
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

Fix "use these mods" context menu option doing broken things with system mods #32235

Merged
merged 5 commits into from
Mar 5, 2025

Conversation

bdach
Copy link
Collaborator

@bdach bdach commented Mar 5, 2025

While the reports claim that the bug "doesn't affect solo", the code is just as wrong, but it's probably backstopped by several other layers of weirdness (like the mod select instance replacement logic, and the song-select touch detector thing).

@bdach bdach added area:song-select area:mods next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! area:daily-challenge labels Mar 5, 2025
@bdach bdach requested a review from a team March 5, 2025 08:51
@bdach bdach self-assigned this Mar 5, 2025
@smoogipoo
Copy link
Contributor

Code looks fine but tests likely need to wait for scores to be loaded.

@smoogipoo smoogipoo merged commit 3f211e3 into ppy:master Mar 5, 2025
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:daily-challenge area:mods area:song-select next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! size/L
Projects
None yet
2 participants