Skip to content

feat(webapi): fetch artist discography via pathfinder GraphQL - #746

Merged
jacksongoode merged 1 commit into
jpochyla:mainfrom
prefixFelix:pr2-artist-discography
Aug 17, 2026
Merged

feat(webapi): fetch artist discography via pathfinder GraphQL#746
jacksongoode merged 1 commit into
jpochyla:mainfrom
prefixFelix:pr2-artist-discography

Conversation

@prefixFelix

Copy link
Copy Markdown
Contributor

Related to #744, split out for review.

Artist albums/singles/compilations/appears-on now come from the pathfinder discography operations on api-partner.spotify.com via partner_auth instead of the REST endpoint, which the API changes broke. Each section is a separate paginated GraphQL query. Also drops the old release-year heuristic that, since the API now returns that section directly.

@prefixFelix

Copy link
Copy Markdown
Contributor Author

We should maybe consider to set some kind of limit for loading appears-on, as the list can be long for bigger artists. Or maybe some kind of load-more-option that the user can trigger.

@jacksongoode jacksongoode left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also looks good. I suggest we do a cleanup pass once we get things working 👍

@jacksongoode

Copy link
Copy Markdown
Collaborator

@prefixFelix Just a few changes to make sure CI/CD pass.

@prefixFelix

Copy link
Copy Markdown
Contributor Author

@prefixFelix Just a few changes to make sure CI/CD pass.

#748

@jacksongoode

Copy link
Copy Markdown
Collaborator

Ok could you rebase now, so we can check the checks?

@jacksongoode

jacksongoode commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

I'll rebase & rerun the builds.

@jacksongoode
jacksongoode force-pushed the pr2-artist-discography branch from 24b06be to b081f7a Compare August 17, 2026 03:20
@jacksongoode
jacksongoode merged commit f8c11b9 into jpochyla:main Aug 17, 2026
9 checks passed
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.

2 participants