[PR #171] [MERGED] 164 subtask create websocket for updating currently playing song information #308

New issue

Closed

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

kerem commented

2026-02-27 19:17:42 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/EddieTheCubeHead/Stagnum/pull/171
Author: @EddieTheCubeHead
Created: 4/1/2024
Status: ✅ Merged
Merged: 4/6/2024
Merged by: @EddieTheCubeHead

Base: master ← Head: 164-subtask-create-websocket-for-updating-currently-playing-song-information

📝 Commits (10+)

03be3c8 164 Base changes
67c3327 Merge branch '66-implement-basic-control-over-pool-playback-randomization' into 164-subtask-create-websocket-for-updating-currently-playing-song-information
fdcb943 164 More merge fixes
73abbf5 164 cache current track data into playback session table in db
e5bba7d 164 Model changes to pool model done
c5c4bb5 164 Websocket working and tested
4766d9a 164 Alembic migration script and test for current playing track return on pool creation
7cad50c Merge branch 'master' into 164-subtask-create-websocket-for-updating-currently-playing-song-information
9d5789d Merge branch 'master' into 164-subtask-create-websocket-for-updating-currently-playing-song-information
e6d4a35 164 Change websockets to accept token from query parameter instead of header

📊 Changes

14 files changed (+291 additions, -86 deletions)

View changed files

➕ server/src/alembic/versions/62ec829ee3f3_caching_current_song_playback_data_in_.py (+32 -0)
📝 server/src/api/common/dependencies.py (+11 -0)
📝 server/src/api/pool/dependencies.py (+86 -29)
📝 server/src/api/pool/helpers.py (+8 -2)
📝 server/src/api/pool/models.py (+1 -0)
📝 server/src/api/pool/routes.py (+25 -12)
📝 server/src/api/pool/tasks.py (+6 -4)
📝 server/src/database/entities.py (+6 -0)
📝 server/test/conftest.py (+6 -0)
📝 server/test/pool_features/conftest.py (+35 -6)
📝 server/test/pool_features/create_pool_features.py (+13 -0)
📝 server/test/pool_features/playback_features.py (+7 -22)
➕ server/test/pool_features/playback_websocket_features.py (+42 -0)
📝 server/test/pool_features/pool_websocket_features.py (+13 -11)

📄 Description

PoolFullContents now has currently playing track data included

Created a websocket (/pool/playback/register_listener) for listening to playback updates. Returns PoolTrack model on playback update (skip or scheduled update on song end)

_{🔄 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/171 **Author:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Created:** 4/1/2024 **Status:** ✅ Merged **Merged:** 4/6/2024 **Merged by:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Base:** `master` ← **Head:** `164-subtask-create-websocket-for-updating-currently-playing-song-information` --- ### 📝 Commits (10+) - [`03be3c8`](https://github.com/EddieTheCubeHead/Stagnum/commit/03be3c82a6f86f450fedbd5ff060462b54c15775) 164 Base changes - [`67c3327`](https://github.com/EddieTheCubeHead/Stagnum/commit/67c33277a1afe209ac8a1bb8d28c004840bce4b5) Merge branch '66-implement-basic-control-over-pool-playback-randomization' into 164-subtask-create-websocket-for-updating-currently-playing-song-information - [`fdcb943`](https://github.com/EddieTheCubeHead/Stagnum/commit/fdcb9436261dd4f3e71f6b723229691eba236a34) 164 More merge fixes - [`73abbf5`](https://github.com/EddieTheCubeHead/Stagnum/commit/73abbf5ed099c5d7421c7913d16a5238ea384461) 164 cache current track data into playback session table in db - [`e5bba7d`](https://github.com/EddieTheCubeHead/Stagnum/commit/e5bba7ddc07281b1b647b263f434b44c567db10e) 164 Model changes to pool model done - [`c5c4bb5`](https://github.com/EddieTheCubeHead/Stagnum/commit/c5c4bb5f6d0c92ef1419248d5cce774aa6493cf0) 164 Websocket working and tested - [`4766d9a`](https://github.com/EddieTheCubeHead/Stagnum/commit/4766d9a22ed86bd0fa76f88a91db63646232884c) 164 Alembic migration script and test for current playing track return on pool creation - [`7cad50c`](https://github.com/EddieTheCubeHead/Stagnum/commit/7cad50cdf43f08d6c8d205abec2b78c963dada1d) Merge branch 'master' into 164-subtask-create-websocket-for-updating-currently-playing-song-information - [`9d5789d`](https://github.com/EddieTheCubeHead/Stagnum/commit/9d5789d6ecd1dd7ccc8067c79a8ea44a59115e10) Merge branch 'master' into 164-subtask-create-websocket-for-updating-currently-playing-song-information - [`e6d4a35`](https://github.com/EddieTheCubeHead/Stagnum/commit/e6d4a357326a738c03f52f77a500ae94d77f1d7d) 164 Change websockets to accept token from query parameter instead of header ### 📊 Changes **14 files changed** (+291 additions, -86 deletions) <details> <summary>View changed files</summary> ➕ `server/src/alembic/versions/62ec829ee3f3_caching_current_song_playback_data_in_.py` (+32 -0) 📝 `server/src/api/common/dependencies.py` (+11 -0) 📝 `server/src/api/pool/dependencies.py` (+86 -29) 📝 `server/src/api/pool/helpers.py` (+8 -2) 📝 `server/src/api/pool/models.py` (+1 -0) 📝 `server/src/api/pool/routes.py` (+25 -12) 📝 `server/src/api/pool/tasks.py` (+6 -4) 📝 `server/src/database/entities.py` (+6 -0) 📝 `server/test/conftest.py` (+6 -0) 📝 `server/test/pool_features/conftest.py` (+35 -6) 📝 `server/test/pool_features/create_pool_features.py` (+13 -0) 📝 `server/test/pool_features/playback_features.py` (+7 -22) ➕ `server/test/pool_features/playback_websocket_features.py` (+42 -0) 📝 `server/test/pool_features/pool_websocket_features.py` (+13 -11) </details> ### 📄 Description PoolFullContents now has currently playing track data included Created a websocket (`/pool/playback/register_listener`) for listening to playback updates. Returns PoolTrack model on playback update (skip or scheduled update on song end) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>