[PR #61] [MERGED] Add browse page #1550

Closed
opened 2026-03-14 15:20:01 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/aome510/spotify-player/pull/61
Author: @aome510
Created: 7/2/2022
Status: Merged
Merged: 7/2/2022
Merged by: @aome510

Base: masterHead: add-browse-page


📝 Commits (10+)

  • cd7bde1 initial cleanup
  • b8a0a6f add ClientRequest::GetBrowseCategories and its handler
  • 994d72f add Command::BrowsePage and its handler
  • 30f92ae add a simple browse page
  • 5052f6f implement command handler for browse page
  • 69d815f make BrowsePageUIState an enum to support different browse page types
  • 209dbd8 wip add category playlists page
  • 6d1907e implement command/event handler for new browse page enum variant
  • cae2920 update page title
  • 556fab2 update README: add BrowsePage command

📊 Changes

13 files changed (+275 additions, -25 deletions)

View changed files

📝 README.md (+1 -0)
📝 spotify_player/src/client/mod.rs (+37 -3)
📝 spotify_player/src/command.rs (+2 -0)
📝 spotify_player/src/config/keymap.rs (+4 -0)
📝 spotify_player/src/event/mod.rs (+13 -0)
📝 spotify_player/src/event/page.rs (+96 -0)
📝 spotify_player/src/event/popup.rs (+3 -0)
📝 spotify_player/src/state/data.rs (+9 -0)
📝 spotify_player/src/state/model.rs (+20 -21)
📝 spotify_player/src/state/ui/page.rs (+23 -1)
📝 spotify_player/src/ui/mod.rs (+1 -0)
📝 spotify_player/src/ui/page.rs (+55 -0)
📝 spotify_player/src/ui/utils.rs (+11 -0)

📄 Description

Changes

  • added a browse page based on Spotify's browse APIs
    • added BrowsePage command (default binding: g b) and its handlers
  • added GetBrowseCategories and GetBrowseCategoryPlaylists client requests along with their handlers
  • cleanup legacy codes

🔄 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/61 **Author:** [@aome510](https://github.com/aome510) **Created:** 7/2/2022 **Status:** ✅ Merged **Merged:** 7/2/2022 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `master` ← **Head:** `add-browse-page` --- ### 📝 Commits (10+) - [`cd7bde1`](https://github.com/aome510/spotify-player/commit/cd7bde1caa6c3d1b5782ad8c12db75a96692f0ed) initial cleanup - [`b8a0a6f`](https://github.com/aome510/spotify-player/commit/b8a0a6f75adb53338aabe59cdbd6cf0f3e0ad610) add `ClientRequest::GetBrowseCategories` and its handler - [`994d72f`](https://github.com/aome510/spotify-player/commit/994d72f397082e448392e89e39477c887479f2d0) add `Command::BrowsePage` and its handler - [`30f92ae`](https://github.com/aome510/spotify-player/commit/30f92aedafbb239bc2f68fad3da8e40a4d210125) add a simple browse page - [`5052f6f`](https://github.com/aome510/spotify-player/commit/5052f6f64bb8f1f3a04c24ee1d2d4518ff95726b) implement command handler for browse page - [`69d815f`](https://github.com/aome510/spotify-player/commit/69d815ff1c8c6413e615c03fa2c10d92a24dbf8b) make `BrowsePageUIState` an enum to support different browse page types - [`209dbd8`](https://github.com/aome510/spotify-player/commit/209dbd81df58cb5464afb22fc29507ba03beaa9b) wip add category playlists page - [`6d1907e`](https://github.com/aome510/spotify-player/commit/6d1907e6d0ffe9284d09526e1748044a0b3aa111) implement command/event handler for new browse page enum variant - [`cae2920`](https://github.com/aome510/spotify-player/commit/cae2920d613e9622691ae087ed6fbcfca766ef16) update page title - [`556fab2`](https://github.com/aome510/spotify-player/commit/556fab275e1b9da7beee46be906a4ae475ada4d8) update README: add `BrowsePage` command ### 📊 Changes **13 files changed** (+275 additions, -25 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+1 -0) 📝 `spotify_player/src/client/mod.rs` (+37 -3) 📝 `spotify_player/src/command.rs` (+2 -0) 📝 `spotify_player/src/config/keymap.rs` (+4 -0) 📝 `spotify_player/src/event/mod.rs` (+13 -0) 📝 `spotify_player/src/event/page.rs` (+96 -0) 📝 `spotify_player/src/event/popup.rs` (+3 -0) 📝 `spotify_player/src/state/data.rs` (+9 -0) 📝 `spotify_player/src/state/model.rs` (+20 -21) 📝 `spotify_player/src/state/ui/page.rs` (+23 -1) 📝 `spotify_player/src/ui/mod.rs` (+1 -0) 📝 `spotify_player/src/ui/page.rs` (+55 -0) 📝 `spotify_player/src/ui/utils.rs` (+11 -0) </details> ### 📄 Description ## Changes - added a browse page based on Spotify's [browse APIs](https://developer.spotify.com/console/browse/) + added `BrowsePage` command (default binding: `g b`) and its handlers - added `GetBrowseCategories` and `GetBrowseCategoryPlaylists` client requests along with their handlers - cleanup legacy codes --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-14 15:20:01 +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/spotify-player#1550
No description provided.