[PR #204] [MERGED] 182 clean users queue on playback start #318

Closed
opened 2026-02-27 19:17:45 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/EddieTheCubeHead/Stagnum/pull/204
Author: @EddieTheCubeHead
Created: 4/7/2024
Status: Merged
Merged: 4/10/2024
Merged by: @EddieTheCubeHead

Base: masterHead: 182-clean-users-queue-on-playback-start


📝 Commits (7)

  • 5114079 182 Add check for user pool being empty before adding song to queue
  • 76e38d5 182 Add test for handling queue not empty exception in scheduled jobs and websockets
  • 3c8e12c Merge branch 'master' into 182-clean-users-queue-on-playback-start
  • 4629a81 182 Fix regressed tests
  • 402a2eb Merge branch 'master' into 182-clean-users-queue-on-playback-start
  • 4d31e1f Merge branch 'master' into 182-clean-users-queue-on-playback-start
  • 59fceb5 Merge branch 'master' into 182-clean-users-queue-on-playback-start

📊 Changes

20 files changed (+181 additions, -52 deletions)

View changed files

📝 server/src/api/auth/dependencies.py (+0 -2)
📝 server/src/api/auth/routes.py (+2 -7)
📝 server/src/api/auth/tasks.py (+0 -6)
📝 server/src/api/pool/dependencies.py (+34 -6)
server/src/api/pool/exceptions.py (+7 -0)
📝 server/src/api/search/dependencies.py (+0 -1)
📝 server/src/api/search/routes.py (+0 -1)
📝 server/src/database/database_connection.py (+0 -1)
📝 server/src/database/entities.py (+2 -2)
📝 server/src/main.py (+1 -0)
📝 server/test/pool_features/add_content_features.py (+0 -1)
📝 server/test/pool_features/conftest.py (+77 -1)
📝 server/test/pool_features/create_pool_features.py (+0 -1)
📝 server/test/pool_features/delete_content_features.py (+0 -1)
📝 server/test/pool_features/playback_features.py (+18 -6)
📝 server/test/pool_features/playback_websocket_features.py (+32 -4)
📝 server/test/pool_features/shared_pool_features.py (+3 -5)
📝 server/test/pool_features/weighted_randomization_features.py (+5 -5)
📝 server/test/search_features/album_search_features.py (+0 -1)
📝 server/test/search_features/general_search_features.py (+0 -1)

📄 Description

Spotify has no route to clear queue

Instead we fetch the queue, and send an error with fix instructions if there are songs in the user queue.


🔄 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/EddieTheCubeHead/Stagnum/pull/204 **Author:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Created:** 4/7/2024 **Status:** ✅ Merged **Merged:** 4/10/2024 **Merged by:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Base:** `master` ← **Head:** `182-clean-users-queue-on-playback-start` --- ### 📝 Commits (7) - [`5114079`](https://github.com/EddieTheCubeHead/Stagnum/commit/51140791cbf44e8b2de04ad07cc540a2b4786738) 182 Add check for user pool being empty before adding song to queue - [`76e38d5`](https://github.com/EddieTheCubeHead/Stagnum/commit/76e38d5b353d1200274e27a73b5784867fc64a4a) 182 Add test for handling queue not empty exception in scheduled jobs and websockets - [`3c8e12c`](https://github.com/EddieTheCubeHead/Stagnum/commit/3c8e12c68a7811b252162a619c2f1a06b470abcd) Merge branch 'master' into 182-clean-users-queue-on-playback-start - [`4629a81`](https://github.com/EddieTheCubeHead/Stagnum/commit/4629a817fa4ec59f96f76dde37a5eabb4d309966) 182 Fix regressed tests - [`402a2eb`](https://github.com/EddieTheCubeHead/Stagnum/commit/402a2ebe3e8736ec4edcf79a8f6570066004ed8e) Merge branch 'master' into 182-clean-users-queue-on-playback-start - [`4d31e1f`](https://github.com/EddieTheCubeHead/Stagnum/commit/4d31e1f0cc080306d428dd6a610244135c5f4413) Merge branch 'master' into 182-clean-users-queue-on-playback-start - [`59fceb5`](https://github.com/EddieTheCubeHead/Stagnum/commit/59fceb593f228c1f90bbc81008d748896dff38be) Merge branch 'master' into 182-clean-users-queue-on-playback-start ### 📊 Changes **20 files changed** (+181 additions, -52 deletions) <details> <summary>View changed files</summary> 📝 `server/src/api/auth/dependencies.py` (+0 -2) 📝 `server/src/api/auth/routes.py` (+2 -7) 📝 `server/src/api/auth/tasks.py` (+0 -6) 📝 `server/src/api/pool/dependencies.py` (+34 -6) ➕ `server/src/api/pool/exceptions.py` (+7 -0) 📝 `server/src/api/search/dependencies.py` (+0 -1) 📝 `server/src/api/search/routes.py` (+0 -1) 📝 `server/src/database/database_connection.py` (+0 -1) 📝 `server/src/database/entities.py` (+2 -2) 📝 `server/src/main.py` (+1 -0) 📝 `server/test/pool_features/add_content_features.py` (+0 -1) 📝 `server/test/pool_features/conftest.py` (+77 -1) 📝 `server/test/pool_features/create_pool_features.py` (+0 -1) 📝 `server/test/pool_features/delete_content_features.py` (+0 -1) 📝 `server/test/pool_features/playback_features.py` (+18 -6) 📝 `server/test/pool_features/playback_websocket_features.py` (+32 -4) 📝 `server/test/pool_features/shared_pool_features.py` (+3 -5) 📝 `server/test/pool_features/weighted_randomization_features.py` (+5 -5) 📝 `server/test/search_features/album_search_features.py` (+0 -1) 📝 `server/test/search_features/general_search_features.py` (+0 -1) </details> ### 📄 Description Spotify has no route to clear queue Instead we fetch the queue, and send an error with fix instructions if there are songs in the user queue. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 19:17:45 +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/Stagnum#318
No description provided.