particl-desktop icon indicating copy to clipboard operation
particl-desktop copied to clipboard

v3 - Feature to delete wallets

Open kangoala opened this issue 4 years ago • 1 comments

Implement an easy way to delete wallets, like already done in v2.

kangoala avatar Dec 25 '20 22:12 kangoala

Leaving this comment here that was made in #1805


Yep, this is something that has been discussed, and as you pointed out, it is something that is missing from the application.

Some background info on why it was omitted for the v3.0 release (for future reference and all):

The particl-market service currently manages its own wallets (or as it refers to them: "identities"). In order to provide various functionality (ensure that it receives smsg destined for specific identity addresses, for example), it also stores the relevant address and some other details for each wallet it manages. Which means, when restoring a marketplace, you'll also need to restore the specific set of wallets it was managing at the time. I believe there's actually a single wallet that needs to be backed up, but regardless, this requires a proper backup and restore process for the particl-market service (which we'lll look at at some point going forward).

By itself, thats not really important. However, consider that the wallet module/app and the market module/app in the application are independent, and don't really know about each other. Which means, if providing for the deleting of a wallet in the wallet app, its possible for the user to delete their market related wallets inadvertently... without any warning from the application because there's no usage meaning associated with each wallet: how does the application know what is the usage for the wallet... all it know is that a deletion request is being made so it actions it accordingly. Unfortunately, such an action results in breaking your marketplace. And without the ability for 'automated' backup and restores of the marketplace at the moment (and with a really finicky manual way to restore a marketplace from a backup), this puts the user in a very bad situation if it happens.


So, what this entails:

  • separate wallets into specific groupings,
  • ensure that any module/app managing their own wallets keep their wallets separate and in a different grouping
  • apps managing wallets can then provide deletion options for those wallets accordingly.

There are means to do this... which builds on from other changes that are necessary or should be done. So yes, this is in the pipeline, just not something we were able to provide with this release.


zaSmilingIdiot avatar Aug 16 '21 09:08 zaSmilingIdiot