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

Move to recycle bin no longer working on Windows 10 #10433

Closed
AssetKid opened this issue Aug 24, 2020 · 3 comments
Closed

Move to recycle bin no longer working on Windows 10 #10433

AssetKid opened this issue Aug 24, 2020 · 3 comments
Labels
more-info-needed The submitter needs to provide more information about the issue

Comments

@AssetKid
Copy link

AssetKid commented Aug 24, 2020

Describe the bug

The Also move this repository to Recycle Bin option no longer works. The file is not moved to the recycle bin.

Version & OS

v2.5.4 | Windows 10 v2004

Steps to reproduce the behavior

  1. Remove...
  2. Select the Also move this repository to Recycle Bin option.
  3. Remove button.

Expected behavior

Folder should be moved to the recycle bin.

Actual behavior

Folder is not moved to the recycle bin.

Screenshots

image

Logs

info: [ui] [BranchPruner] No branches to prune.

Additional context

Potential Cause: #10143

@niik
Copy link
Member

niik commented Aug 24, 2020

Hey @AssetKid, thanks for opening this and sorry to hear you're having problems deleting repositories. I tried this myself and I was unable to reproduce it. Can you try creating a brand new repository from within the app and then removing it (including moving it to the Recycle Bin) and see if it's still reproducible?

I'm wondering if you're hitting #8333. I.e. there's an open file within the repository and the removal fails because of that. Due to what seems to be a bug in electron we're not able to detect when a delete fails and show an appropriate error message.

remove-test

@niik niik added the more-info-needed The submitter needs to provide more information about the issue label Aug 24, 2020
@AssetKid
Copy link
Author

It is working as expected after creating a repo from within the app.

@niik
Copy link
Member

niik commented Aug 25, 2020

Thanks for confirming @AssetKid. I'm going to close this since we have a PR open to resolve this as soon as electron has a fix out on their side

@niik niik closed this as completed Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
more-info-needed The submitter needs to provide more information about the issue
Projects
None yet
Development

No branches or pull requests

2 participants