Skip to content

fix(spotify/artist): include all music#21593

Open
jameschubbuck wants to merge 1 commit intoDIYgod:masterfrom
jameschubbuck:master
Open

fix(spotify/artist): include all music#21593
jameschubbuck wants to merge 1 commit intoDIYgod:masterfrom
jameschubbuck:master

Conversation

@jameschubbuck
Copy link
Copy Markdown

Involved Issue / 该 PR 相关 Issue

Close #

Example for the Proposed Route(s) / 路由地址示例

/spotify/artist

New RSS Route Checklist / 新 RSS 路由检查表

  • New Route / 新的路由
  • Anti-bot or rate limit / 反爬/频率限制
    • If yes, do your code reflect this sign? / 如果有, 是否有对应的措施?
  • Date and time / 日期和时间
    • Parsed / 可以解析
    • Correct time zone / 时区正确
  • New package added / 添加了新的包
  • Puppeteer

Note / 说明

By default the artist endpoint only returns albums and omits other types. It would be better to include them.

@github-actions github-actions bot added the route label Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Auto Review

No clear rule violations found in the current diff.

@github-actions github-actions bot added auto: not ready to review Users can't get the RSS feed output according to automated testing results and removed auto: not ready to review Users can't get the RSS feed output according to automated testing results labels Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Successfully generated as following:

http://localhost:1200/spotify/artist - Failed ❌
HTTPError: Response code 503 (Service Unavailable)

Error Message:<br/>NotFoundError:
Route: /spotify/artist
Full Route: /spotify/artist
Node Version: v24.14.1
Git Hash: 85a609e5

},
});
const itemsResponse = await ofetch(`https://api.spotify.com/v1/artists/${id}/albums`, {
const itemsResponse = await ofetch(`https://api.spotify.com/v1/artists/${id}/albums?include_groups=album,single,ep,appears_on,compilation`, {
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.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Oh - my bad. Do you know how to include EPs? I didn't see them show up in any other field so assumed that it must exist.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto: not ready to review Users can't get the RSS feed output according to automated testing results route

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants