[PR #551] Update PlaylistItem API to new API #534

New issue

Open

opened 2026-02-27 20:25:05 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 20:25:05 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ramsayleung/rspotify/pull/551
Author: @Manishearth
Created: 2/23/2026
Status: 🔄 Open

Base: master ← Head: new-api

📝 Commits (2)

6347a30 Update PlaylistItem API to new API
fdc7b4b fix some of the writeable APIs

📊 Changes

4 files changed (+9 additions, -3 deletions)

View changed files

📝 rspotify-model/src/playlist.rs (+2 -0)
📝 rspotify-model/src/track.rs (+4 -0)
📝 src/clients/base.rs (+1 -1)
📝 src/clients/oauth.rs (+2 -2)

📄 Description

Description

Makes progress on https://github.com/ramsayleung/rspotify/issues/550 .

This is an example of a way to upgrade the API. Note that some fields have also been deprecated; I have marked them as such and added serde(default) where needed. It may be worth removing all of them in a breaking change.

Motivation and Context

The code doesn't work; this makes it work. See the linked issue, Spotify dropped a major API change in the last moment.

Dependencies

List any dependencies that are required for this change.

Type of change

Bug fix (non-breaking change which fixes an issue)

How has this been tested?

After this change I'm able to successfully call playlist_items on a playlist full of tracks and have it work. I have not tried this with playlists containing episodes.

A holistic approach is probably best here, but I'm content fixing the APIs that I need for now.

Is this change properly documented?

shrug

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ramsayleung/rspotify/pull/551 **Author:** [@Manishearth](https://github.com/Manishearth) **Created:** 2/23/2026 **Status:** 🔄 Open **Base:** `master` ← **Head:** `new-api` --- ### 📝 Commits (2) - [`6347a30`](https://github.com/ramsayleung/rspotify/commit/6347a30d788fb9529b13cea3bf5e881997ebee10) Update PlaylistItem API to new API - [`fdc7b4b`](https://github.com/ramsayleung/rspotify/commit/fdc7b4ba93d396761ade9cfb7843dc0b73bd98b8) fix some of the writeable APIs ### 📊 Changes **4 files changed** (+9 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `rspotify-model/src/playlist.rs` (+2 -0) 📝 `rspotify-model/src/track.rs` (+4 -0) 📝 `src/clients/base.rs` (+1 -1) 📝 `src/clients/oauth.rs` (+2 -2) </details> ### 📄 Description ## Description Makes progress on https://github.com/ramsayleung/rspotify/issues/550 . This is an example of a way to upgrade the API. Note that some fields have also been deprecated; I have marked them as such and added `serde(default)` where needed. It _may_ be worth removing all of them in a breaking change. ## Motivation and Context The code doesn't work; this makes it work. See the linked issue, Spotify dropped a major API change in the last moment. ## Dependencies List any dependencies that are required for this change. ## Type of change - [x] Bug fix (non-breaking change which fixes an issue) ## How has this been tested? After this change I'm able to successfully call `playlist_items` on a playlist full of tracks and have it work. I have not tried this with playlists containing episodes. A holistic approach is probably best here, but I'm content fixing the APIs that I need for now. ## Is this change properly documented? _shrug_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>