[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 blueb...@eyepaste.com changed: What|Removed |Added CC||blueb...@eyepaste.com --- Comment #15 from blueb...@eyepaste.com --- For the Kubuntu 18.04.2 LTS users who got here due Google: Use Michael Tunnell's manual solution, since Kubuntu 18.04.2 is still stuck at 5.12 at the time of writing this comment. -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 Nate Graham changed: What|Removed |Added Version Fixed In||5.13 -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 Ivan Čukić changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED|RESOLVED Latest Commit||https://commits.kde.org/pla ||sma-vault/eadb8c0689fd6d8bf ||55a758471ce75cb9b982849 --- Comment #14 from Ivan Čukić --- Git commit eadb8c0689fd6d8bf55a758471ce75cb9b982849 by Ivan Čukić. Committed on 10/04/2018 at 21:22. Pushed by ivan into branch 'master'. Added vault deletion through the configuration dialogue A +89 -0asynqt/private/wrappers/kjob_p.h [License: LGPL] A +51 -0asynqt/wrappers/kjob.h [License: LGPL] M +3-2common/vaultinfo.h M +2-0kded/CMakeLists.txt M +3-3kded/engine/backend_p.h M +2-1kded/engine/commandresult.h M +19 -7kded/engine/fusebackend_p.cpp M +3-3kded/engine/fusebackend_p.h M +52 -37 kded/engine/vault.cpp M +1-1kded/engine/vault.h M +45 -1kded/service.cpp M +3-0kded/service.h M +18 -4kded/ui/dialogdsl.cpp M +2-0kded/ui/dialogdsl.h M +2-0kded/ui/directorychooserwidget.cpp M +6-0kded/ui/namechooserwidget.cpp M +24 -10 kded/ui/vaultconfigurationwizard.cpp A +95 -0kded/ui/vaultdeletionwidget.cpp [License: GPL (v2/3)] C +18 -38 kded/ui/vaultdeletionwidget.h [from: kded/ui/namechooserwidget.cpp - 057% similarity] A +65 -0kded/ui/vaultdeletionwidget.ui M +1-1plasma/vaultsmodel.cpp https://commits.kde.org/plasma-vault/eadb8c0689fd6d8bf55a758471ce75cb9b982849 -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #13 from vinnywright --- (In reply to Ivan Čukić from comment #12) > It will be a part of the release after the LTS. Didn't want to add new > features in this one - just polish and bug fixes. cool ,,,just curious ,,,and some people were asking about it on the forum I hang out on https://www.kubuntuforums.net/showthread.php/72484-Where-is-Plasma-Vault?p=410320#post410320 thank you for the quick reply :) -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 naraesk changed: What|Removed |Added CC|m...@naraesk.eu | -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #12 from Ivan Čukić --- It will be a part of the release after the LTS. Didn't want to add new features in this one - just polish and bug fixes. -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #11 from vinnywright --- so has their been any work on this ? it's been quite a wile now ,, -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 Bastian Beischer changed: What|Removed |Added CC||bastian.beischer@rwth-aache ||n.de -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 Nate Graham changed: What|Removed |Added Keywords||usability -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 Simon Andric changed: What|Removed |Added CC||simonandr...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #10 from vinnywright --- (In reply to Nate Graham from comment #9) > Once the feature is implemented, yes. cool ,,,other than that it seems to work nicely. -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #9 from Nate Graham --- Once the feature is implemented, yes. -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 vinnywright changed: What|Removed |Added CC||charles.v.wri...@gmail.com --- Comment #8 from vinnywright --- well nice to see this is going to be fixed ,,as I could find no way to delete/remove a test vault . the user IS told that forgetting the password will result in non recoverable data loss so I think just destroying the vault and data IS the option . will this be "fixed" as an update to the installed plasma-vault? -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #7 from Nate Graham --- That's a really great idea, cryptodude. -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 cryptod...@libertymail.net changed: What|Removed |Added CC||cryptod...@libertymail.net --- Comment #6 from cryptod...@libertymail.net --- Maybe you can just hard delete the mountpoint and move the vault backing dir into the kde trash. Then you can have an 'undelete' button as well inside of your value UI. -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 naraesk changed: What|Removed |Added CC||m...@naraesk.eu --- Comment #5 from naraesk --- I'd suggest that to delete a vault the user needs to enter the name of the vault instead of simpley clicking an Ok button. This minimizes the risk of deleting the wrong vault by accident. -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #4 from Ivan Čukić --- Ok. I'm convinced :) -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #3 from Michael Tunnell --- @Ivan Čukić > I'm leaning towards having 'dismantle' just unregister (your third step) the > vault and force the user to delete the directories manually (with this stated > in the UI so that the user does not have any surprises). I think there should be an offer of Dismantle and Delete. Dismantling doing as you suggest where it just removes the entry but the data is fine. Deleting would be a full delete of entry and files/folders. @Nate Graham > The UI should make it clear that deleting a vault with information in it is a > destructive action, and we should provide an undo for mistakes. This could be approached in the way Dolphin has Delete & Shift+Delete for permanent deletion. > we ought to treat users as adults capable of making their own decision and > whose time is valuable. This *exactly this*. So often I've seen projects insist that something can't be done because of fear that "it is too destructive" which of course is replied by the user with "yes, that is exactly the point of why I want it." -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #2 from Nate Graham --- I don't agree that users should have to use the file system or a terminal to delete vaults. The UI should make it clear that deleting a vault with information in it is a destructive action, and we should provide an undo for mistakes. Beyond that, we ought to treat users as adults capable of making their own decision (so we should let them delete vaults) and whose time is valuable (so they shouldn't have to muck around in the file system and learn how to do it manually). -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 Ivan Čukić changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 --- Comment #1 from Ivan Čukić --- I agree it is important. And it will be released with 5.12. The reason why it is not released is that I'm a bit torn about what 'dismantling' should do. I'm leaning towards having 'dismantle' just unregister (your third step) the vault and force the user to delete the directories manually (with this stated in the UI so that the user does not have any surprises). My main rule with PV is to approach everything from the data-safety point first, and then user convenience. Thoughts? (p.s. I didn't expect people immediately wanting to delete vaults -- I forgot that people will want to test how the things work :) ) -- You are receiving this mail because: You are watching all bug changes.
[Plasma Vault] [Bug 385444] Add Ability to Delete a Vault via GUI
https://bugs.kde.org/show_bug.cgi?id=385444 Nate Graham changed: What|Removed |Added CC||pointedst...@zoho.com -- You are receiving this mail because: You are watching all bug changes.
