[PR #17] [MERGED] Integrate Spotify Recommendations API #1514

New issue

Closed

opened 2026-03-14 15:18:07 +03:00 by kerem · 0 comments

kerem commented

2026-03-14 15:18:07 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/aome510/spotify-player/pull/17
Author: @aome510
Created: 10/22/2021
Status: ✅ Merged
Merged: 10/22/2021
Merged by: @aome510

Base: master ← Head: integrate-recommendations-API

📝 Commits (10+)

5c1a25c add Action::BrowseRecommendation
992e5a3 move the ui::PageState on-change handler from client module to ui module
25e76b2 handle parsing action::BrowseRecommendations
53127fd handle GetRecommendations client request
b69a893 implement rendering function for recommendation window
438145a add a unimplemented TODO for handling key sequence for the recommendation window
d758c8e handle user input key sequence for recommendation page
b9c4453 change the number of tracks returned by the recommandation API to 50
37697ea update roadmap
211a9e0 update the is_active condition for top track subwindow in the artist context window

📊 Changes

14 files changed (+433 additions, -198 deletions)

View changed files

📝 README.md (+2 -2)
📝 doc/config.md (+1 -1)
📝 spotify_player/src/client/handlers.rs (+0 -70)
📝 spotify_player/src/client/mod.rs (+50 -0)
📝 spotify_player/src/command.rs (+3 -2)
📝 spotify_player/src/config/keymap.rs (+1 -1)
📝 spotify_player/src/event/mod.rs (+22 -0)
📝 spotify_player/src/event/popup.rs (+16 -3)
📝 spotify_player/src/event/window.rs (+52 -16)
📝 spotify_player/src/main.rs (+2 -1)
📝 spotify_player/src/state/player.rs (+24 -1)
📝 spotify_player/src/state/ui.rs (+7 -1)
📝 spotify_player/src/ui/context.rs (+26 -84)
📝 spotify_player/src/ui/mod.rs (+227 -16)

📄 Description

Brief description of changes

integrate Spotify recommendation API
rename command::PlayContext to command::PlayRandom
move the code that updates state::WindowState based on current state::PageState from the client module to the ui module
small refactors for ui module

_{🔄 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/aome510/spotify-player/pull/17 **Author:** [@aome510](https://github.com/aome510) **Created:** 10/22/2021 **Status:** ✅ Merged **Merged:** 10/22/2021 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `master` ← **Head:** `integrate-recommendations-API` --- ### 📝 Commits (10+) - [`5c1a25c`](https://github.com/aome510/spotify-player/commit/5c1a25c04df2089b19cfe40274848a34e23ea2e4) add `Action::BrowseRecommendation` - [`992e5a3`](https://github.com/aome510/spotify-player/commit/992e5a3731fd53cc79fb494380f8fb86d49d4273) move the `ui::PageState` on-change handler from `client` module to `ui` module - [`25e76b2`](https://github.com/aome510/spotify-player/commit/25e76b2664ad9f4889b5cd56b15432a31752e077) handle parsing `action::BrowseRecommendations` - [`53127fd`](https://github.com/aome510/spotify-player/commit/53127fdf6bdf1d50aac414fa0fd46226fe53927f) handle `GetRecommendations` client request - [`b69a893`](https://github.com/aome510/spotify-player/commit/b69a893d9cd88ae3ea2ba706bf9545b76e6976e4) implement rendering function for recommendation window - [`438145a`](https://github.com/aome510/spotify-player/commit/438145ae3083548d1b3b6374a231a33da98d9cb0) add a `unimplemented` TODO for handling key sequence for the recommendation window - [`d758c8e`](https://github.com/aome510/spotify-player/commit/d758c8edbf980e537f161678dcf12af2c0c869d0) handle user input key sequence for recommendation page - [`b9c4453`](https://github.com/aome510/spotify-player/commit/b9c44534dd486448dd1c5417270ab216a99f48c7) change the number of tracks returned by the recommandation API to `50` - [`37697ea`](https://github.com/aome510/spotify-player/commit/37697eacf94fc26b2daee38b0354ca1e1a00d2dd) update `roadmap` - [`211a9e0`](https://github.com/aome510/spotify-player/commit/211a9e0e244eef623afb0d74d104131b3c7acbe1) update the `is_active` condition for top track subwindow in the artist context window ### 📊 Changes **14 files changed** (+433 additions, -198 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+2 -2) 📝 `doc/config.md` (+1 -1) 📝 `spotify_player/src/client/handlers.rs` (+0 -70) 📝 `spotify_player/src/client/mod.rs` (+50 -0) 📝 `spotify_player/src/command.rs` (+3 -2) 📝 `spotify_player/src/config/keymap.rs` (+1 -1) 📝 `spotify_player/src/event/mod.rs` (+22 -0) 📝 `spotify_player/src/event/popup.rs` (+16 -3) 📝 `spotify_player/src/event/window.rs` (+52 -16) 📝 `spotify_player/src/main.rs` (+2 -1) 📝 `spotify_player/src/state/player.rs` (+24 -1) 📝 `spotify_player/src/state/ui.rs` (+7 -1) 📝 `spotify_player/src/ui/context.rs` (+26 -84) 📝 `spotify_player/src/ui/mod.rs` (+227 -16) </details> ### 📄 Description ## Brief description of changes - integrate Spotify [recommendation API](https://developer.spotify.com/console/get-recommendations/) - rename `command::PlayContext` to `command::PlayRandom` - move the code that updates `state::WindowState` based on current `state::PageState` from the `client` module to the `ui` module - small refactors for `ui` module --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>