[PR #24] [CLOSED] Add support for deleting from playlists and some improvements to how data is sent with a request #729

Closed
opened 2026-02-28 00:01:22 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/spotipy-dev/spotipy/pull/24
Author: @drsm79
Created: 8/19/2014
Status: Closed

Base: masterHead: master


📝 Commits (5)

📊 Changes

5 files changed (+166 additions, -87 deletions)

View changed files

📝 .gitignore (+1 -0)
📝 README.md (+9 -7)
requirements.txt (+1 -0)
📝 spotipy/__init__.py (+108 -62)
📝 spotipy/oauth2.py (+47 -18)

📄 Description

This PR has a few things beyond the two issues mentioned in the commits (#21 & #22):

  • Add a requirements.txt
  • Some PEP8 clean up.
  • Handle request data for POST/PUT/DELETE more generally
  • Rely on request's raise_for_status to identify issues with a request.
  • Report send specific error messages to the SpotifyException, taken from the response json
  • Tweak some documentation

🔄 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/spotipy-dev/spotipy/pull/24 **Author:** [@drsm79](https://github.com/drsm79) **Created:** 8/19/2014 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (5) - [`026ba0f`](https://github.com/spotipy-dev/spotipy/commit/026ba0f5dd0919592bc83c2135011b204c178a33) pep8 - [`2b1d333`](https://github.com/spotipy-dev/spotipy/commit/2b1d33306445ad6d49b37bf5a8a667af81346db7) add requirements.txt - [`63ba8d8`](https://github.com/spotipy-dev/spotipy/commit/63ba8d848c1911167ceecb0147f365f7b423e213) 201 is a valid response (issue #21) - [`aa8961b`](https://github.com/spotipy-dev/spotipy/commit/aa8961bc1c05a7da25965447a6178ec1d2fc2086) Update doc string for add tracks - [`eff7c70`](https://github.com/spotipy-dev/spotipy/commit/eff7c701f6a7dce9a5ad214b04e91bee77a17d6c) Add delete from playlist (issue #22) ### 📊 Changes **5 files changed** (+166 additions, -87 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -0) 📝 `README.md` (+9 -7) ➕ `requirements.txt` (+1 -0) 📝 `spotipy/__init__.py` (+108 -62) 📝 `spotipy/oauth2.py` (+47 -18) </details> ### 📄 Description This PR has a few things beyond the two issues mentioned in the commits (#21 & #22): - Add a requirements.txt - Some PEP8 clean up. - Handle request data for POST/PUT/DELETE more generally - Rely on request's raise_for_status to identify issues with a request. - Report send specific error messages to the SpotifyException, taken from the response json - Tweak some documentation --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 00:01:22 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/spotipy#729
No description provided.