Skip to content

Comments

when deleting mods, delete them to the Recycle Bin#314

Merged
JohnAFernandez merged 3 commits intoKnossosNET:mainfrom
Goober5000:delete_to_recycle_bin
Mar 19, 2025
Merged

when deleting mods, delete them to the Recycle Bin#314
JohnAFernandez merged 3 commits intoKnossosNET:mainfrom
Goober5000:delete_to_recycle_bin

Conversation

@JohnAFernandez
Copy link
Contributor

This needs cross-platform testing, but I'm on board with the changes.

@Goober5000 Goober5000 marked this pull request as draft March 11, 2025 05:13
@Goober5000
Copy link
Contributor Author

According to ShivanSpS, it produces an error on Linux, but there should be a straightforward way around that. I need a little bit more information and then I'll try a fix. I'll put it in draft in the meantime.

@Goober5000 Goober5000 marked this pull request as ready for review March 13, 2025 18:13
@Goober5000
Copy link
Contributor Author

Goober5000 commented Mar 13, 2025

With the latest change, ShivanSpS reports a successful fall back to regular deletion.

Copy link
Member

@Shivansps Shivansps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works correctly. it recycles on windows and it is deleted permantly on Linux as a fallback

Well done.

@JohnAFernandez JohnAFernandez merged commit 933a818 into KnossosNET:main Mar 19, 2025
@Goober5000 Goober5000 deleted the delete_to_recycle_bin branch March 20, 2025 00:30
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.

3 participants