plamere / SmarterPlaylists

web app for creating sophisticated playlists
384 stars 58 forks source link

Malformed playlist field leads to confusing 'total' error #31

Open emallove opened 2 years ago

emallove commented 2 years ago

The playlist URI field must be in spotify:playlist:<ID> format, e.g., the below HTTP URL format will not work:

spotify playlist -- in URL form

But the error message caused by the malformed playlist field is confusing (red div with the text 'total'):

confounding total red error message up top

Alyx611 commented 1 year ago

I had the same issue. I fixed it by changing the uri field from this fromat "https://open.spotify.com/playlist/**PLAYLIST_ID**?si=123456789" to this "spotify:user:spotify:playlist:PLAYLIST_ID"

flawedgun commented 1 year ago

I had the same issue. I fixed it by changing the uri field from this fromat "https://open.spotify.com/playlist/**PLAYLIST_ID**?si=123456789" to this "spotify:user:spotify:playlist:PLAYLIST_ID"

Agree; that's the way to format the URI. The issue @emallove is raising is that the error message is unhelpful for users trying to resolve the error (i.e., there's nothing to indicate that the URI was incorrectly formatted, the issue could be anything).

larralapid commented 10 months ago

Also just found this out after 10+ years of being a Spotify user: When you right click on a playlist in the desktop app and navigate your cursor to the "share" menu item, if you have the option butten held down, instead of copying the URL to the playlist, you can instead copy the Spotify URI.

beacharoni commented 10 months ago

Thanks :)

Am So., 29. Okt. 2023 um 23:58 Uhr schrieb larra @.***>:

Also just found this out after 10+ years of being a Spotify user: When you right click on a playlist in the desktop app and navigate your cursor to the "share" menu item, if you have the option butten held down, instead of copying the URL to the playlist, you can instead copy the Spotify URI.

— Reply to this email directly, view it on GitHub https://github.com/plamere/SmarterPlaylists/issues/31#issuecomment-1784255913, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAY5BLRLF2WUJSU5PMOPWULYB3NQZAVCNFSM5V53L322U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCNZYGQZDKNJZGEZQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>