Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI not updated after playlist deletion #758

Open
orontee opened this issue Aug 15, 2021 · 2 comments
Open

UI not updated after playlist deletion #758

orontee opened this issue Aug 15, 2021 · 2 comments
Labels

Comments

@orontee
Copy link
Contributor

orontee commented Aug 15, 2021

Describe the bug
Deletion of a playlist removes the corresponding .m3u8 file but the playlist is still displayed in the UI.

To Reproduce

  1. Create a playlist with local files : a .m3u8 file is created under /var/lib/mopidy/m3u or equivalent
  2. Right click on the play list to open its contextual menu
  3. Select "delete"
  4. The .m3u8 file is deleted but the playlist is still there in the UI

Expected behavior

I expect the playlist to be removed from the UI.

Screenshots
[If applicable, add screenshots to help explain your problem.]

Environment details:

  • OS: Debian 11
  • Browser: Firefox
  • Iris version: 3.58.0

Additional context
One can delete the playlist from the page displaying the playlist content. An analogous bug happens when deleting from there: The UI is not updated and the browser isn't redirected to another page (which one?).

@orontee orontee added the bug label Aug 15, 2021
@jaedb
Copy link
Owner

jaedb commented Aug 21, 2021

This is a see-saw bug that is caused by #678 by not removing the deleted playlist from the app state, nor redirecting.

@jaedb
Copy link
Owner

jaedb commented Mar 20, 2022

The playlist is removed from the library, but remains in the local (browser) database. This is to ensure that Spotify playlists can be 'undeleted' (which is effectively just re-following it). Returning to the library will show it is now removed, but you can still revisit it by using Back in the browser window.

A solution I think will serve both scenarios is keeping it as-is, but flagging the playlist as deleted so it is clear that the playlist on screen has been deleted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants