-
Notifications
You must be signed in to change notification settings - Fork 257
Documentation for Mobileclient.get_promoted_songs()
is misleading.
#637
Comments
I was pretty convinced in the past that it wasn't - see #282 - but maybe it changed? I'm hesitant to change it because a) I'm lazy, and b) I don't want to address another ticket if someone finds a counterexample in their library. I think I'd be more willing to just deprecate it, to be honest. |
How about a compromise? I'll submit a PR that deprecates this function and makes a new one called get_top_songs() with the documentation
Does that sound good? |
Sure, that sounds fine. Thanks! |
…t::get_top_songs` Fixes simon-weber#637
…t::get_top_songs` Fixes simon-weber#637
Just figured I'd add that for me, the results from Edit: So different devices show different songs for the "Thumbs Up" playlist, and even Edit 2: You get more similar results if you combine the songs from |
The current list of liked songs is only populated with those that happen to have been enumerated, leaving out songs that have been marked as thumbs up but are not in the library or a playlist. This patch explicitly walks the list from "get_top_songs()", which contains some of the songs that have been marked as thumbs up. The result is that my liked songs list matches what is shown on my phone. See simon-weber/gmusicapi#637 for more information. Signed-off-by: Palmer Dabbelt <[email protected]>
As I discovered while reading #633, the documentation for Mobileclient.get_promoted_songs() is misleading. The documentation as written reads:
But it seems that it returns the complete list of store songs that have a positively-rated, in the exact order that "Thumbs-up" auto-playlist lists them. It even includes songs that have been explicitly added to the library.
The text was updated successfully, but these errors were encountered: