mirror of
https://github.com/ramsayleung/rspotify.git
synced 2026-04-26 16:05:53 +03:00
[PR #529] [MERGED] fix(model): handle null images on FullPlaylist model #517
Labels
No labels
Stale
bug
discussion
enhancement
good first issue
good first issue
help wanted
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/rspotify#517
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/ramsayleung/rspotify/pull/529
Author: @lazulit3
Created: 6/18/2025
Status: ✅ Merged
Merged: 6/19/2025
Merged by: @ramsayleung
Base:
master← Head:fix-fullplaylist-null-images📝 Commits (1)
5ee88ddfix(model): handle null images on FullPlaylist model📊 Changes
1 file changed (+1 additions, -0 deletions)
View changed files
📝
rspotify-model/src/playlist.rs(+1 -0)📄 Description
Description
Fixes a deserialization error that occurs when spotify returns a playlist with
"images": nullwhen deserializing intoFullPlaylist.This was fixed for
SimplifiedPlaylist, but we need the fix forFullPlaylistas well:Motivation and Context
I am using
playlist()to fetch a playlist by its ID, and my playlist has no tracks (and therefor no images), and I'm encountering this deserialization error that I'd like to fix.Dependencies
None.
Type of change
Please delete options that are not relevant.
How has this been tested?
I've pointed my project at my fork (on this branch) and verified that this resolves the deserialization failure.
Is this change properly documented?
Not documented.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.