[PR #399] [MERGED] Correctly handle playlist unfollowing #801

Closed
opened 2026-02-28 14:53:24 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Rigellute/spotify-tui/pull/399
Author: @slumber
Created: 4/13/2020
Status: Merged
Merged: 4/16/2020
Merged by: @Rigellute

Base: masterHead: playlists-fix


📝 Commits (1)

  • e8ed863 Correctly handle playlist unfollowing

📊 Changes

2 files changed (+14 additions, -1 deletions)

View changed files

📝 src/app.rs (+13 -0)
📝 src/handlers/search_results.rs (+1 -1)

📄 Description

How to reproduce the problem:

  1. Follow any playlist, it will appear in the left frame "Playlists"
  2. Search for any other playlist and select it in the corresponding frame
  3. Press 'D'
    The one that you were following will disappear.

A simpler case - have no playlists followed and press 'D' - the application will panic with an out of bounds.

Unfollowing playlists from search results should be handled separately.


🔄 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/Rigellute/spotify-tui/pull/399 **Author:** [@slumber](https://github.com/slumber) **Created:** 4/13/2020 **Status:** ✅ Merged **Merged:** 4/16/2020 **Merged by:** [@Rigellute](https://github.com/Rigellute) **Base:** `master` ← **Head:** `playlists-fix` --- ### 📝 Commits (1) - [`e8ed863`](https://github.com/Rigellute/spotify-tui/commit/e8ed863b4946bbe88f5c95959d6b394c38a35de3) Correctly handle playlist unfollowing ### 📊 Changes **2 files changed** (+14 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/app.rs` (+13 -0) 📝 `src/handlers/search_results.rs` (+1 -1) </details> ### 📄 Description **How to reproduce the problem:** 1) Follow any playlist, it will appear in the left frame "Playlists" 2) Search for any other playlist and select it in the corresponding frame 3) Press 'D' The one that you were following will disappear. A simpler case - have no playlists followed and press 'D' - the application will panic with an out of bounds. Unfollowing playlists from search results should be handled separately. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 14:53:24 +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-tui#801
No description provided.