rspotify icon indicating copy to clipboard operation
rspotify copied to clipboard

PlaylistItem.track possibly shouldn't be wrapped in Option

Open kangalio opened this issue 2 years ago • 2 comments

At least when returned from Get Playlist, the field is never absent:

Screenshot_20230519_014641

Link

I'm not sure which other endpoints return payloads containing PlaylistItem - if PlaylistItem.track is optional in any of those, we probably shouldn't duplicate PlaylistItem and instead add a doc comment to BaseClient.playlist(): "the [`PlaylistItem::track`] field in this response is always Some".

kangalio avatar May 18 '23 23:05 kangalio

Message to comment on stale issues. If none provided, will not mark issues stale

github-actions[bot] avatar Nov 15 '23 02:11 github-actions[bot]

Another valid inprovement suggestion lost to the relentless bug report steamroller called stale bot :'(

kangalio avatar Nov 30 '23 10:11 kangalio