ppy / osu

rhythm is just a *click* away!
https://osu.ppy.sh
MIT License
15.09k stars 2.23k forks source link

Playlist leaderboard scores display incorrect date #19292

Open frenzibyte opened 2 years ago

frenzibyte commented 2 years ago

Discussed in https://github.com/ppy/osu/discussions/19291

Originally posted by **Flamiii** July 22, 2022 ![image](https://user-images.githubusercontent.com/74088315/180353862-eb30cdee-7baf-481f-8319-a69e5166e236.png) So currently in playlists, when you hover over the '-' character next to a user's country flag, it will display a date of 1 January 0001. I'm assuming that this is a placeholder for now, but I think that it would be cool if the '-' character was replaced with the last time the user played in this specific playlist so that you can get a better idea of how recently someone played in a playlist. I'm not sure if this is possible to do just yet but it's something that I think would be nice to see in the future.
frenzibyte commented 2 years ago

Actually, it appears the playlist leaderboard scores never receive the date from the API? I don't see any property relating to date in the JSON response:

       {
            "accuracy": 0.9927924718185444,
            "attempts": 40,
            "completed": 9,
            "pp": 0,
            "room_id": 213518,
            "total_score": 9670543,
            "user_id": 8116659,
            "user": {
                "avatar_url": "https://a.ppy.sh/8116659?1651349097.gif",
                "country_code": "DE",
                "default_group": "default",
                "id": 8116659,
                "is_active": true,
                "is_bot": false,
                "is_deleted": false,
                "is_online": false,
                "is_supporter": true,
                "last_visit": "2022-07-21T22:23:36+00:00",
                "pm_friends_only": false,
                "profile_colour": null,
                "username": "criller",
                "country": {
                    "code": "DE",
                    "name": "Germany"
                }
            }
        },