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

Add FileOperationsPlugin #3550

Closed
wants to merge 1 commit into from

Conversation

mkalinski
Copy link

Check-list

  • There is a linked issue discussing the motivations for this feature or bugfix: Improve docs around DnD to copy files and rename to move #3549
  • Unit tests have been added where possible
  • I've added / updated documentation for any user-facing features: individual plugins don't seem to have a dedicated documentation section
  • Performance seems to be comparable or better than current master

What this change is adding / fixing

This change is adding a songsmenu plugin that provides two menu options: Copy and Move.

These options open a file chooser to select destination, and then copy or move the selected songs to the destination.

Moving / copying is done in a separate thread.

A window is opened with a progress bar, log of each file copied / moved and any exceptions that might occur, and a button to abort the copy / move.

@declension
Copy link
Member

Did you try doing via Custom Commands ? It seems like it should be doable

@mkalinski
Copy link
Author

@declension It's technically doable via Custom Commands but it's cumbersome. The destination can only be passed as parameter, which needs to be manually typed as text input every time.

There are good alternatives to this plugin proposed in #3549 though.

@declension
Copy link
Member

Closing in favour of #3459

@declension declension closed this Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants