-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Update Simplified ranking criteria
#14145
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
Conversation
|
Someone else has edited same files as you did. Please check their changes in case they conflict with yours: |
MChecaH
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thoughts on adding a "Last updated" date somewhere. Could make it easy to know when the page is due for a change at a first glance. Probably somewhere around the intro, to also let users know how outdated the article is.
|
At that point I'd do so for all RCs, thoughts? |
|
Sssssure. Just saw more value having it on the sub-article at first. But on second thought I do think having an update date on all criterias would be in line with other similar articles. |
|
I also saw that idea at ppy/osu-web #12630 Shouldn't that updated time be added automatically through bot or something else? Further they can be added to the meta part of articles by bot and be rendered by osu-web with better position and better style through some css |
in theory this would be ideal yes, but i guess that's something to be addressed elsewhere (e.g. not in the scope of this pr) |
MChecaH
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed with Hivie that adding the last update is out of scope for this particular PR. Can be added in a posterior update.
🧹
brings a couple things up to speed with main RC