-
Notifications
You must be signed in to change notification settings - Fork 22
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
[Feat ure] Remove container feature #39
Comments
ok, it seems reasonable to me.
..............................................................
*ANDREA CHIUMENTI*
*Partner*
Tel. +39 039 9639600
Mob. +39 348 8063547
Mail. ***@***.***
Via Gramsci, 10 - 20900 Monza
P.Iva 09154830963
red.software.systems
…On Thu, Aug 24, 2023 at 5:51 AM hhoao ***@***.***> wrote:
Can you add the ability to remove containers feature?
—
Reply to this email directly, view it on GitHub
<#39>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAESDCUVSRHRPPQHQUC4UVLXW3FTRANCNFSM6AAAAAA34MFGB4>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Ciao ragazzi! I love the reduced user interface of the extension! 😍 ... but, yes, the "Remove" container feature is missing. Do you want a PR for this to be added? |
Ciao Peter,
We could reintroduce it ... with many warings, when there was in the
original version, I accidentally removed a container. On the other hand,
there are still tasks that I prefer to do via console.
I think that deleting from the extension would be perhaps too easy. There
is also a very nice extension for VS code if you are interested.
(So basically I could also add a delete op, but with an alert dialog kind
of "Type DELETE if you really want to delete container X" is necessary)
Andrea
..............................................................
*ANDREA CHIUMENTI*
*Partner*
Tel. +39 039 9639600
Mob. +39 348 8063547
Mail. ***@***.***
Via Gramsci, 10 - 20900 Monza
P.Iva 09154830963
red.software.systems
…On Thu, Sep 28, 2023 at 2:58 PM Peter Bittner ***@***.***> wrote:
Ciao ragazzi!
I love the reduced user interface of the extension! 😍 ... but, yes, the
"Remove" container feature is missing.
Do you want a PR for this to be added?
—
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAESDCTZ25ZM2XIXRBNOS2DX4VYA3ANCNFSM6AAAAAA34MFGB4>
.
You are receiving this because you commented.Message ID:
***@***.***
com>
|
Sure! A confirmation dialog would be totally fine. But keep it simple: A confirmation dialog with an OK/Cancel or Yes/No choice is sufficient. After all, what's the problem when an image is deleted? You should be able to either rebuild it (when you're the developer) or download it again (when the image is provided by someone else). Ideally, the operation should be possible only on stopped containers. |
Can you add the ability to remove containers feature?
The text was updated successfully, but these errors were encountered: