delete forked state environment from env explorer #5600
Conversation
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| @@ -687,6 +687,7 @@ export class Blockchain extends Plugin { | |||
| } | |||
|
|
|||
| removeProvider(name) { | |||
| if (this.pinnedProviders.includes(name)) this.emit('shouldRemoveProviderFromUdapp', name, this.getProviderObjByName(name)) | |||
There was a problem hiding this comment.
this is already emitted by on('environmentExplorer', 'providerUnpinned'. Could you explain why you need this here?
There was a problem hiding this comment.
Without this, it doesn't remove provider from dropdown even after deletion. A similar check is there in addProvider too
Aniket-Engg
changed the title
|
the following question could arise, should we leave the delete & fork buttons after a VM has been forked - so you can fork a fork or delete it. With this PR, at least if you can find the Environment Explorer, you can delete. And as I use this functionality, it makes sense that the delete button should not be available to a fork like you can to delete the VM state. |
This will allow to delete forked state provider from Environment Explorer
For forked state, it will show this button:
It will also delete the state file from workspace.