Skip to content

Feat/delete volume#3

Merged
glefer merged 1 commit into
mainfrom
feat/delete-volume
May 26, 2025
Merged

Feat/delete volume#3
glefer merged 1 commit into
mainfrom
feat/delete-volume

Conversation

@glefer

@glefer glefer commented May 26, 2025

Copy link
Copy Markdown
Owner

This feature introduces the ability to remove Docker volumes that do not have any containers attached. It enhances the functionality of the docker-volume-analyzer by allowing users to manage and clean up unused volumes more effectively.

Screen Capture_select-area_20250526145159

Key Changes

New Feature:

Added functionality to check if a volume has attached containers before allowing its deletion.
If a volume has no attached containers, it can now be safely removed.

Error Handling:

If a user attempts to delete a volume with attached containers, an error message is displayed, preventing accidental deletion.

Confirmation Dialog:

A confirmation screen is displayed before deleting a volume, asking the user to confirm the action.

UI Updates:

Added a modal confirmation screen (ConfirmationScreen) for deletion actions.
Added an error screen (ErrorScreen) to notify users when a volume cannot be deleted.

@glefer glefer force-pushed the feat/delete-volume branch 2 times, most recently from d2631cd to 747bd96 Compare May 26, 2025 12:50
@github-actions

github-actions Bot commented May 26, 2025

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/docker_volume_analyzer
   __init__.py00100% 
   docker_client.py310100% 
   errors.py30100% 
   main.py90100% 
   tui.py1080100% 
   volume_manager.py240100% 
TOTAL1750100% 

@glefer glefer requested a review from Copilot May 26, 2025 12:58
@glefer glefer marked this pull request as ready for review May 26, 2025 13:00

This comment was marked as outdated.

@glefer glefer requested a review from Copilot May 26, 2025 13:01

This comment was marked as resolved.

@glefer glefer force-pushed the feat/delete-volume branch from 747bd96 to 5eb0c73 Compare May 26, 2025 13:11
@glefer glefer merged commit 4ca380b into main May 26, 2025
2 checks passed
@glefer glefer deleted the feat/delete-volume branch May 26, 2025 13:14
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

Successfully merging this pull request may close these issues.

2 participants