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

Implement rank milestone medals #235

Merged
merged 2 commits into from
Mar 21, 2024
Merged

Conversation

bdach
Copy link
Collaborator

@bdach bdach commented Mar 12, 2024

Closes ppy/osu#27581.

I did mention this wasn't implemented in #195 (comment), but it was a long PR thread so it might have gotten missed.

Not sure if this should be merged imminently while the yo-yo effect with ranks shifting because of the bonus pp discrepancy is still ongoing. I believe this should no longer be a concern.

@peppy
Copy link
Member

peppy commented Mar 12, 2024

Not sure if this should be merged imminently while the yo-yo effect with ranks shifting because of the bonus pp discrepancy is still ongoing.

I hope to fix that this week.

@peppy peppy self-requested a review March 21, 2024 06:20
@peppy peppy merged commit b0f65b0 into ppy:master Mar 21, 2024
3 checks passed
@bdach bdach deleted the rank-milestone-medals branch March 21, 2024 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

'I Can See The Top' medal not awarded
2 participants