Skip to content

Add topSongsByArtistId extension#259

Merged
sentriz merged 1 commit into
mainfrom
feat/gettopsongs-artist-id
Jul 24, 2026
Merged

Add topSongsByArtistId extension#259
sentriz merged 1 commit into
mainfrom
feat/gettopsongs-artist-id

Conversation

@sentriz

@sentriz sentriz commented Jul 17, 2026

Copy link
Copy Markdown
Member

getTopSongs currently only accepts an artist name. Which is ambiguous since two
artists can share the same name, so the server has to guess which one the client
meant

Passing an artist ID removes the ambiguity

  • A server backed by something like ListenBrainz (MBID based) can resolve the
    exact artist directly, disambiguating same named artists
  • Even a string-based backend like last.fm benefits. The server can resolve the
    artist from the ID it already knows, then fetch/filter that artist's top songs
    precisely instead of round-tripping a name

@netlify

netlify Bot commented Jul 17, 2026

Copy link
Copy Markdown

Deploy Preview for opensubsonic ready!

Name Link
🔨 Latest commit 60c21aa
🔍 Latest deploy log https://app.netlify.com/projects/opensubsonic/deploys/6a634ea2a221c600089ca489
😎 Deploy Preview https://deploy-preview-259--opensubsonic.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@Tolriq

Tolriq commented Jul 17, 2026

Copy link
Copy Markdown
Member

For the record this also fit #258 ;) filter artistId sort by topSongs and is one more argument for it ;)

@sentriz
sentriz requested a review from a team July 21, 2026 12:59
@sentriz
sentriz force-pushed the feat/gettopsongs-artist-id branch from a5d98ad to 60c21aa Compare July 24, 2026 11:38
@sentriz
sentriz requested review from a team July 24, 2026 11:46
@sentriz
sentriz merged commit 4c1819f into main Jul 24, 2026
4 checks passed
@sentriz
sentriz deleted the feat/gettopsongs-artist-id branch July 24, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants