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

Remove a contract #41

Open
zoedberg opened this issue Nov 28, 2023 · 3 comments
Open

Remove a contract #41

zoedberg opened this issue Nov 28, 2023 · 3 comments
Assignees
Milestone

Comments

@zoedberg
Copy link
Contributor

We think it would be useful to have a method to "de-register"/"forget" an asset, or in other words to remove data related to an asset from the stash. Currently AFAIK there is no way to do this. We think there could be cases where a user doesn't care anymore about an asset, he is ok with burning it, but to reduce stash data consumption (and possibly improve performance) it would be nice to have a method that allows this.
This is not urgent but a nice to have, IMO it can be done after 0.11 testing.

@dr-orlovsky
Copy link
Member

I think we should allow removal of an asset only if the user doesn't own any state under that asset. Unlike in blockchain, undesirable airdrops are not possible (just do not accept the consignment), thus it should be enough.

@fedsten
Copy link
Member

fedsten commented Nov 28, 2023

There may be cases where the wallet accept an incoming transfer because it is valid, but then the user wants to remove the asset because the attachment or the asset itself may be illegal or regulatory risky to own in his jurisdiction, or other cases where the user was initially happy to own the asset, but then a change in regulation made it a liability to keep owning it.

At the end there are many ways for a wallet to lose the assets of the users, so we should trust wallet developer to use this feature in a safe manner.

@dr-orlovsky dr-orlovsky added this to the v0.10.x milestone Dec 30, 2023
@dr-orlovsky dr-orlovsky self-assigned this Dec 30, 2023
@dr-orlovsky dr-orlovsky changed the title remove an asset Remove a contract Dec 30, 2023
@zoedberg
Copy link
Contributor Author

@dr-orlovsky can we move this to a later milestone? 0.12 for example should be ok

@dr-orlovsky dr-orlovsky modified the milestones: v0.10.x, v0.12.0 Mar 22, 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

No branches or pull requests

3 participants