[PR #324] [CLOSED] Split '?' when extracting ID with HTTP URLs #855

New issue

Closed

opened 2026-02-28 00:02:09 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:02:09 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/spotipy-dev/spotipy/pull/324
Author: @ritiek
Created: 10/2/2018
Status: ❌ Closed

Base: master ← Head: fix-split-http-url

📝 Commits (10+)

90d6622 Re-order imports and specify UTF-8 encoding header in all base modules and package initialiser (ref. https://www.python.org/dev/peps/pep-0008/#imports, https://www.python.org/dev/peps/pep-0008/#source-file-encoding) + add __all__ attribute
5f47ae2 Update spotify package initialiser
8cc601f Remove relative imports in spotify package initialiser
816dac8 Miscellaneous fixes - clean up tests
613d99a Further clean up of tests
578b46e Renaming some test modules: authtests.py -> test_auth.py, authtests2.py -> test_auth2.py, client_credential_tests.py -> test_client_credentials.py + update tox.ini
2c5d55d Update tests - make client credentials env. vars list a commonly accessible dictionary in spotipy/util.py
d3dc229 Minor docs fix
db60668 Update client.py (add playlist endpoint)
608d9ee Merge pull request #1 from chunksong/chunksong-update-client(add-playlist-endpoint)

📊 Changes

40 files changed (+742 additions, -474 deletions)

View changed files

📝 .gitignore (+1 -0)
📝 CHANGES.txt (+24 -16)
📝 README.md (+1 -0)
📝 deploy (+2 -2)
📝 docs/index.rst (+54 -47)
📝 examples/artist_discography.py (+1 -1)
📝 examples/contains_a_saved_track.py (+1 -1)
📝 examples/my_playlists.py (+1 -1)
📝 examples/my_top_artists.py (+3 -3)
📝 examples/my_top_tracks.py (+5 -5)
📝 examples/read_a_playlist.py (+1 -1)
📝 examples/remove_specific_tracks_from_playlist.py (+1 -2)
📝 examples/remove_tracks_from_playlist.py (+1 -2)
📝 examples/show_album.py (+3 -1)
📝 examples/show_artist.py (+3 -1)
📝 examples/show_artist_top_tracks.py (+3 -1)
📝 examples/show_related.py (+4 -1)
📝 examples/show_track_info.py (+3 -1)
📝 examples/simple0.py (+4 -1)
📝 examples/simple1.py (+5 -4)

...and 20 more files

📄 Description

Closes #323.

_{🔄 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/324 **Author:** [@ritiek](https://github.com/ritiek) **Created:** 10/2/2018 **Status:** ❌ Closed **Base:** `master` ← **Head:** `fix-split-http-url` --- ### 📝 Commits (10+) - [`90d6622`](https://github.com/spotipy-dev/spotipy/commit/90d66222fd6e1af28a1fe80eb3a21ccf6064e96a) Re-order imports and specify UTF-8 encoding header in all base modules and package initialiser (ref. https://www.python.org/dev/peps/pep-0008/#imports, https://www.python.org/dev/peps/pep-0008/#source-file-encoding) + add `__all__` attribute - [`5f47ae2`](https://github.com/spotipy-dev/spotipy/commit/5f47ae29bd94a70adc9fb046ee8727c1dfb1441f) Update spotify package initialiser - [`8cc601f`](https://github.com/spotipy-dev/spotipy/commit/8cc601fe53adb8138accd61b3f1acdcb06488e72) Remove relative imports in `spotify` package initialiser - [`816dac8`](https://github.com/spotipy-dev/spotipy/commit/816dac861f812ae079a3cbae1b89433c7d82774e) Miscellaneous fixes - clean up tests - [`613d99a`](https://github.com/spotipy-dev/spotipy/commit/613d99a031fc7956673faf1cf3b999e1b3d03c0f) Further clean up of tests - [`578b46e`](https://github.com/spotipy-dev/spotipy/commit/578b46e8cbead0db3b5465880d926ebb7b09ca0c) Renaming some test modules: `authtests.py` -> `test_auth.py`, `authtests2.py` -> `test_auth2.py`, `client_credential_tests.py` -> `test_client_credentials.py` + update `tox.ini` - [`2c5d55d`](https://github.com/spotipy-dev/spotipy/commit/2c5d55d2e0d82f76e33a533c7d0e2c302f07e0af) Update tests - make client credentials env. vars list a commonly accessible dictionary in `spotipy/util.py` - [`d3dc229`](https://github.com/spotipy-dev/spotipy/commit/d3dc2290cf6292cab0ac5ed4c8600ab8298ecff2) Minor docs fix - [`db60668`](https://github.com/spotipy-dev/spotipy/commit/db6066847b4c5c1fda063c96e2af42d89a4dcf4c) Update client.py (add playlist endpoint) - [`608d9ee`](https://github.com/spotipy-dev/spotipy/commit/608d9ee7c1cb6640ac72a00e28cdf8117195cc16) Merge pull request #1 from chunksong/chunksong-update-client(add-playlist-endpoint) ### 📊 Changes **40 files changed** (+742 additions, -474 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -0) 📝 `CHANGES.txt` (+24 -16) 📝 `README.md` (+1 -0) 📝 `deploy` (+2 -2) 📝 `docs/index.rst` (+54 -47) 📝 `examples/artist_discography.py` (+1 -1) 📝 `examples/contains_a_saved_track.py` (+1 -1) 📝 `examples/my_playlists.py` (+1 -1) 📝 `examples/my_top_artists.py` (+3 -3) 📝 `examples/my_top_tracks.py` (+5 -5) 📝 `examples/read_a_playlist.py` (+1 -1) 📝 `examples/remove_specific_tracks_from_playlist.py` (+1 -2) 📝 `examples/remove_tracks_from_playlist.py` (+1 -2) 📝 `examples/show_album.py` (+3 -1) 📝 `examples/show_artist.py` (+3 -1) 📝 `examples/show_artist_top_tracks.py` (+3 -1) 📝 `examples/show_related.py` (+4 -1) 📝 `examples/show_track_info.py` (+3 -1) 📝 `examples/simple0.py` (+4 -1) 📝 `examples/simple1.py` (+5 -4) _...and 20 more files_ </details> ### 📄 Description Closes #323. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>