Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing paMissing parameter in "areFollowingPlaylist" #483

Open
Luca-Tamburo opened this issue Mar 14, 2023 · 3 comments · May be fixed by #486
Open

Missing paMissing parameter in "areFollowingPlaylist" #483

Luca-Tamburo opened this issue Mar 14, 2023 · 3 comments · May be fixed by #486

Comments

@Luca-Tamburo
Copy link

In the documentation, the first parameter of "areFollowingPlaylist" is not specified.
In the doc it is written that "playlistId", and an array of usernames suffice, but in the implementation it is specified (as the first parameter), the "userId".

  • Documentation version: areFollowingPlaylist(playlistId, [followerIds])

  • Implementation version: areFollowingPlaylist(userId, playlistId, [followerIds])

Could you please update the documentation?

@pdlima
Copy link

pdlima commented Apr 28, 2023

Looks like the endpoint structure itself might've changed.

This is the current endpoint from the Spotify docs: GET /playlists/{playlist_id}/followers/contains
This is how it's being implemented here: GET /users/{user_id/playlists/{playlist_id}/followers/contains

Link to the Spotify docs: https://developer.spotify.com/documentation/web-api/reference/check-if-user-follows-playlist

@enzoferey
Copy link

I'm opening a PR to cover this change 👍🏻

@Abdullah-Malik
Copy link

If you are looking to write an app with a newly written and updated SDK, take a look - https://github.com/Abdullah-Malik/spotified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants