[PR #200] [MERGED] Add docs for new features and improve CLI/daemon codes #1600

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

📋 Pull Request Information

Original PR: https://github.com/aome510/spotify-player/pull/200
Author: @aome510
Created: 5/17/2023
Status: Merged
Merged: 5/18/2023
Merged by: @aome510

Base: masterHead: add-docs-for-new-features


📝 Commits (10+)

  • 75aca98 setup default values for cache-folder and config-folder CLI options
  • b85f9d5 highlight the default configuration files in config document
  • 9b37bb2 add client_port doc and example config
  • c95d9e8 add CLI commands and Daemon documentations
  • 2758a7a cleanup
  • 91cb7ca Merge remote-tracking branch 'origin/master' into add-docs-for-new-features
  • 3ba3a76 handle sending/receiving response for all CLI commands
  • 0f6168a use PlayerReqest to handle playback CLI commands
  • 874449c move like command from playback subcommand to be a general subcommand
  • 9740667 implement authenticate CLI command

📊 Changes

11 files changed (+243 additions, -205 deletions)

View changed files

📝 README.md (+33 -2)
📝 docs/config.md (+5 -4)
📝 examples/app.toml (+1 -0)
📝 spotify_player/Cargo.toml (+1 -1)
📝 spotify_player/src/auth.rs (+8 -8)
📝 spotify_player/src/cli/client.rs (+84 -139)
📝 spotify_player/src/cli/commands.rs (+15 -10)
📝 spotify_player/src/cli/handlers.rs (+46 -17)
📝 spotify_player/src/cli/mod.rs (+8 -2)
📝 spotify_player/src/client/mod.rs (+2 -2)
📝 spotify_player/src/main.rs (+40 -20)

📄 Description

Changes

  • add docs for CLI commands and daemon
  • add authenticate and like CLI commands (replace playback like command)
  • update handling logic for playback commands to rely on theclient::Client::handle_player_request function
  • improve errors/logs reporting

🔄 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/200 **Author:** [@aome510](https://github.com/aome510) **Created:** 5/17/2023 **Status:** ✅ Merged **Merged:** 5/18/2023 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `master` ← **Head:** `add-docs-for-new-features` --- ### 📝 Commits (10+) - [`75aca98`](https://github.com/aome510/spotify-player/commit/75aca9896e5db1d9b079e7230ade3d3a01866abd) setup default values for `cache-folder` and `config-folder` CLI options - [`b85f9d5`](https://github.com/aome510/spotify-player/commit/b85f9d5677f76819b5ed3af7588af9320bfab89d) highlight the default configuration files in config document - [`9b37bb2`](https://github.com/aome510/spotify-player/commit/9b37bb26a362a923cf9a2fb584ec57feaa698445) add `client_port` doc and example config - [`c95d9e8`](https://github.com/aome510/spotify-player/commit/c95d9e8889b9e4019d1b862aa34994dd62d9982c) add CLI commands and Daemon documentations - [`2758a7a`](https://github.com/aome510/spotify-player/commit/2758a7a5bb481aff9e7470527bccc46d48f475b5) cleanup - [`91cb7ca`](https://github.com/aome510/spotify-player/commit/91cb7ca878bdfec0b2bac537e995d7b70022ffbb) Merge remote-tracking branch 'origin/master' into add-docs-for-new-features - [`3ba3a76`](https://github.com/aome510/spotify-player/commit/3ba3a7613191bc17d3887b8ade699924b0d4842e) handle sending/receiving response for all CLI commands - [`0f6168a`](https://github.com/aome510/spotify-player/commit/0f6168a48c9af051b0bde4f35cec6beb633a39f8) use `PlayerReqest` to handle playback CLI commands - [`874449c`](https://github.com/aome510/spotify-player/commit/874449c23161e9875de0de629767045a24886a9a) move like command from playback subcommand to be a general subcommand - [`9740667`](https://github.com/aome510/spotify-player/commit/974066754c9ed489c69575872869e06b250fe017) implement authenticate CLI command ### 📊 Changes **11 files changed** (+243 additions, -205 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+33 -2) 📝 `docs/config.md` (+5 -4) 📝 `examples/app.toml` (+1 -0) 📝 `spotify_player/Cargo.toml` (+1 -1) 📝 `spotify_player/src/auth.rs` (+8 -8) 📝 `spotify_player/src/cli/client.rs` (+84 -139) 📝 `spotify_player/src/cli/commands.rs` (+15 -10) 📝 `spotify_player/src/cli/handlers.rs` (+46 -17) 📝 `spotify_player/src/cli/mod.rs` (+8 -2) 📝 `spotify_player/src/client/mod.rs` (+2 -2) 📝 `spotify_player/src/main.rs` (+40 -20) </details> ### 📄 Description ## Changes - add docs for **CLI commands** and **daemon** - add `authenticate` and `like` CLI commands (replace `playback like` command) - update handling logic for `playback` commands to rely on the`client::Client::handle_player_request` function - improve errors/logs reporting --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-14 15:22:43 +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#1600
No description provided.